github-mirror/framework
2
0
Fork 0
mirror of https://github.com/flarum/framework.git synced 2024-11-25 17:50:38 +08:00
Code Issues Actions 46 Packages Projects Releases Wiki Activity

PERF: Don't redefine subscription menu items on each redraw

Browse Source
This commit is contained in:
Toby Zerner 2015-11-03 16:04:39 +10:30
parent 7dab0d76c0
commit 1e49c46d51
1 changed files with 24 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
extensions/subscriptions/js/forum/src/components/SubscriptionMenu.js
View File

@ -5,6 +5,29 @@ import icon from 'flarum/helpers/icon';
import SubscriptionMenuItem from 'flarum/subscriptions/components/SubscriptionMenuItem'; import SubscriptionMenuItem from 'flarum/subscriptions/components/SubscriptionMenuItem';
export default class SubscriptionMenu extends Component { export default class SubscriptionMenu extends Component {
init() {
this.options = [
{
subscription: false,
icon: 'star-o',
label: app.translator.trans('flarum-subscriptions.forum.sub_controls.not_following_button'),
description: app.translator.trans('flarum-subscriptions.forum.sub_controls.not_following_text')
},
{
subscription: 'follow',
icon: 'star',
label: app.translator.trans('flarum-subscriptions.forum.sub_controls.following_button'),
description: app.translator.trans('flarum-subscriptions.forum.sub_controls.following_text')
},
{
subscription: 'ignore',
icon: 'eye-slash',
label: app.translator.trans('flarum-subscriptions.forum.sub_controls.ignoring_button'),
description: app.translator.trans('flarum-subscriptions.forum.sub_controls.ignoring_text')
}
];
}
view() { view() {
const discussion = this.props.discussion; const discussion = this.props.discussion;
const subscription = discussion.subscription(); const subscription = discussion.subscription();
@ -28,27 +51,6 @@ export default class SubscriptionMenu extends Component {
// no default // no default
} }
const options = [
{
subscription: false,
icon: 'star-o',
label: app.translator.trans('flarum-subscriptions.forum.sub_controls.not_following_button'),
description: app.translator.trans('flarum-subscriptions.forum.sub_controls.not_following_text')
},
{
subscription: 'follow',
icon: 'star',
label: app.translator.trans('flarum-subscriptions.forum.sub_controls.following_button'),
description: app.translator.trans('flarum-subscriptions.forum.sub_controls.following_text')
},
{
subscription: 'ignore',
icon: 'eye-slash',
label: app.translator.trans('flarum-subscriptions.forum.sub_controls.ignoring_button'),
description: app.translator.trans('flarum-subscriptions.forum.sub_controls.ignoring_text')
}
];
return ( return (
<div className="Dropdown ButtonGroup SubscriptionMenu"> <div className="Dropdown ButtonGroup SubscriptionMenu">
{Button.component({ {Button.component({
@ -63,7 +65,7 @@ export default class SubscriptionMenu extends Component {
</button> </button>
<ul className="Dropdown-menu dropdown-menu Dropdown-menu--right"> <ul className="Dropdown-menu dropdown-menu Dropdown-menu--right">
{options.map(props => { {this.options.map(props => {
props.onclick = this.saveSubscription.bind(this, discussion, props.subscription); props.onclick = this.saveSubscription.bind(this, discussion, props.subscription);
props.active = subscription === props.subscription; props.active = subscription === props.subscription;