Style tweaks

Follows strictly jshint and jscs rules.

.jscsrc

@@ -68,6 +68,7 @@
     ],
     "safeContextKeyword": [
         "self",
+        "context",
         "view"
     ],
     "validateLineBreaks": "LF",

.meteor/packages

@@ -9,6 +9,7 @@ meteor-platform
 accounts-password
 kenton:accounts-sandstorm
 service-configuration
+useraccounts:core
 useraccounts:unstyled
 
 # Compilers
@@ -50,4 +51,3 @@ mquandalle:jquery-textcomplete
 peerlibrary:blaze-components
 reactive-var
 seriousm:emoji-continued
-useraccounts:core

README.md

@@ -20,6 +20,6 @@ made with [Meteor](https://www.meteor.com).
 [Our roadmap is self-hosted on LibreBoard][roadmap]
 
 [travis-status]: https://travis-ci.org/libreboard/libreboard.svg
-[travis-link]: https://travis-ci.org/libreboard/libreboard.svg
+[travis-link]: https://travis-ci.org/libreboard/libreboard
 [thumbnail]: http://i.imgur.com/IIdHUmW.png
 [roadmap]: http://libreboard.com/boards/MeSsFJaSqeuo9M6bs/libreboard-roadmap

client/components/cards/details.js

@@ -53,7 +53,7 @@ BlazeComponent.extendComponent({
 
         event.preventDefault();
       },
-      'click .js-cancel-edit': function(event, t) {
+      'click .js-cancel-edit': function() {
         // remove editing hide.
         $('.editing').removeClass('editing');
       },
@@ -64,8 +64,8 @@ BlazeComponent.extendComponent({
             $set: {
               title: title
             }
-          }, function (err, res) {
-            if (!err) $('.editing').removeClass('editing');
+          }, function(err) {
+            if (! err) $('.editing').removeClass('editing');
           });
         }
 
@@ -77,7 +77,7 @@ BlazeComponent.extendComponent({
             description: t.find('#desc').value
           }
         }, function(err) {
-          if (!err) $('.editing').removeClass('editing');
+          if (! err) $('.editing').removeClass('editing');
         });
         event.preventDefault();
       },

client/components/lists/body.js

@@ -61,7 +61,8 @@ BlazeComponent.extendComponent({
           var list = $('#js-list-' + this.data()._id);
           var nextList = list[isReverse ? 'prev' : 'next']('.js-list').get(0) ||
             $('.js-list:' + (isReverse ? 'last' : 'first')).get(0);
-          var nextListComponent = BlazeComponent.getComponentForElement(nextList);
+          var nextListComponent =
+                                BlazeComponent.getComponentForElement(nextList);
 
           // XXX Get the real position
           var position = 'bottom';

client/components/lists/main.js

@@ -1,4 +1,4 @@
-ListComponent = BlazeComponent.extendComponent({
+BlazeComponent.extendComponent({
   template: function() {
     return 'list';
   },
@@ -27,19 +27,19 @@ ListComponent = BlazeComponent.extendComponent({
     if (Meteor.user().isBoardMember()) {
       var $cards = this.$('.js-minicards');
       $cards.sortable({
-        connectWith: ".js-minicards",
+        connectWith: '.js-minicards',
         tolerance: 'pointer',
         appendTo: '.js-lists',
-        helper: "clone",
+        helper: 'clone',
         items: '.js-minicard:not(.placeholder, .hide, .js-composer)',
         placeholder: 'minicard placeholder',
-        start: function (event, ui) {
+        start: function(event, ui) {
           $('.minicard.placeholder').height(ui.item.height());
           Popup.close();
         },
         stop: function(event, ui) {
-          // To attribute the new index number, we need to get the dom element of
-          // the previous and the following card -- if any.
+          // To attribute the new index number, we need to get the dom element
+          // of the previous and the following card -- if any.
           var cardDomElement = ui.item.get(0);
           var prevCardDomElement = ui.item.prev('.js-minicard').get(0);
           var nextCardDomElement = ui.item.next('.js-minicard').get(0);
@@ -57,7 +57,7 @@ ListComponent = BlazeComponent.extendComponent({
 
       Utils.liveEvent('mouseover', function($el) {
         $el.find('.js-member-droppable').droppable({
-          hoverClass: "draggable-hover-card",
+          hoverClass: 'draggable-hover-card',
           accept: '.js-member',
           drop: function(event, ui) {
             var memberId = Blaze.getData(ui.draggable.get(0)).userId;
@@ -67,7 +67,7 @@ ListComponent = BlazeComponent.extendComponent({
         });
 
         $el.find('.js-member-droppable').droppable({
-          hoverClass: "draggable-hover-card",
+          hoverClass: 'draggable-hover-card',
           accept: '.js-label',
           drop: function(event, ui) {
             var labelId = Blaze.getData(ui.draggable.get(0))._id;

client/components/lists/menu.js

@@ -3,8 +3,8 @@ Template.listActionPopup.events({
     // XXX We need a better API and architecture here. See
     // https://github.com/peerlibrary/meteor-blaze-components/issues/19
     var listDom = document.getElementById('js-list-' + this._id);
-    var listComponent = Blaze.getView(listDom).templateInstance().get('component');
-    listComponent.openForm();
+    var listInstance = Blaze.getView(listDom).templateInstance();
+    listInstance.get('component').openForm();
     Popup.close();
   },
   'click .js-list-subscribe': function() {},

client/components/main/helpers.js

@@ -45,7 +45,7 @@ Blaze.Template.registerHelper('mentions', new Template('mentions', function() {
 
   var mentionRegex = /\B@(\w*)/gi;
   var currentMention, knowedUser, href, linkClass, linkValue, link;
-  while (currentMention = mentionRegex.exec(content)) {
+  while (!! (currentMention = mentionRegex.exec(content))) {
 
     knowedUser = _.findWhere(knowedUsers, { username: currentMention[1] });
     if (! knowedUser)
@@ -53,7 +53,9 @@ Blaze.Template.registerHelper('mentions', new Template('mentions', function() {
 
     linkValue = [' ', at, knowedUser.username];
     href = Router.url('Profile', { username: knowedUser.username });
-    linkClass = 'atMention' + (knowedUser.userId === Meteor.userId() ? ' me' : '');
+    linkClass = 'atMention';
+    if (knowedUser.userId === Meteor.userId())
+      linkClass += ' me';
     link = HTML.A({ href: href, 'class': linkClass }, linkValue);
 
     content = content.replace(currentMention[0], Blaze.toHTML(link));