Correct lint warnings.

ericf · Dec 5, 2012 · db60213 · db60213
1 parent 8533c70
commit db60213
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/public/js/app.js b/public/js/app.js
@@ -61,13 +61,17 @@ PhotosNearMe = Y.Base.create('photosNearMe', Y.App, [], {
         };
 
         // Update the title of the browser window.
-        doc && (doc.title = this.titleTemplate({place: placeData}));
+        if (doc) {
+            doc.title = this.titleTemplate({place: placeData});
+        }
 
         content = this.headerTemplate({place: placeData});
 
         container.removeClass('loading').one('#header').setHTML(content);
         // Delay adding `located` class so the CSS transitions run.
-        place.isNew() || Y.later(1, container, 'addClass', 'located');
+        if (!place.isNew()) {
+            Y.later(0, container, 'addClass', 'located');
+        }
 
         return this;
     },

diff --git a/public/js/models/photo.js b/public/js/models/photo.js
@@ -81,7 +81,7 @@ Photo = Y.Base.create('photo', Y.Model, [Y.ModelSync.YQL], {
         delete data.pageUrl;
 
         return data;
-    },
+    }
 
 }, {
 

diff --git a/public/js/views/lightbox.js b/public/js/views/lightbox.js
@@ -78,7 +78,7 @@ LightboxView = Y.Base.create('lightboxView', Y.View, [], {
 
     toggleInfo: function (e) {
         var infoNode = this.infoNode,
-            visible  = !(infoNode.getStyle('display') === 'none');
+            visible  = infoNode.getStyle('display') !== 'none';
 
         if (visible) {
             infoNode.hide('fadeOut');