diff options
author | Maxime Quandalle <maxime@quandalle.com> | 2015-10-22 04:02:12 +0200 |
---|---|---|
committer | Maxime Quandalle <maxime@quandalle.com> | 2015-10-22 18:13:12 +0200 |
commit | aa974aa54ab6e5b7db7450206d12b44ffb3a0306 (patch) | |
tree | b501ee16fa630e19b95d99ffc3984e30301e096a /client/components/sidebar/sidebar.js | |
parent | c6b12dc5ada1b37d759796fefe0dbc5b327f130c (diff) | |
download | wekan-aa974aa54ab6e5b7db7450206d12b44ffb3a0306.tar.gz wekan-aa974aa54ab6e5b7db7450206d12b44ffb3a0306.tar.bz2 wekan-aa974aa54ab6e5b7db7450206d12b44ffb3a0306.zip |
Prefer ES5 methods over underscore utilities
Since 07cc454 (ie the switch to Meteor 1.2) we includes the `es5-shim`
polyfill to support methods like `Array.prototype.forEach` in a
consistent way across all supported browsers (IE8+).
MDG recently released a blog post recommending the use of these native
methods instead of underscore [0]. We know follow this recommendation.
This commit also favor some ES6 features (argument defaults,
destructing assignment) in places where we didn’t use them.
[0]: http://info.meteor.com/blog/es2015-get-started
Diffstat (limited to 'client/components/sidebar/sidebar.js')
-rw-r--r-- | client/components/sidebar/sidebar.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/client/components/sidebar/sidebar.js b/client/components/sidebar/sidebar.js index 72866055..ccb9f2f5 100644 --- a/client/components/sidebar/sidebar.js +++ b/client/components/sidebar/sidebar.js @@ -95,10 +95,10 @@ BlazeComponent.extendComponent({ events() { // XXX Hacky, we need some kind of `super` const mixinEvents = this.getMixin(Mixins.InfiniteScrolling).events(); - return mixinEvents.concat([{ + return [...mixinEvents, { 'click .js-toggle-sidebar': this.toggle, 'click .js-back-home': this.setView, - }]); + }]; }, }).register('sidebar'); |