(chore) Add ESLint config and fix errors

franksoong · Mar 11, 2017 · 6d5ff72 · 6d5ff72
1 parent f1b5484
commit 6d5ff72
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 11 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -0,0 +1,20 @@
+module.exports = {
+    "env": {
+        "browser": true
+    },
+    "extends": "eslint:recommended",
+    "rules": {
+        "indent": [
+            "error",
+            4
+        ],
+        "quotes": [
+            "error",
+            "single"
+        ],
+        "semi": [
+            "error",
+            "always"
+        ]
+    }
+};
diff --git a/src/baguetteBox.js b/src/baguetteBox.js
@@ -288,15 +288,15 @@
 
     function keyDownHandler(event) {
         switch (event.keyCode) {
-            case 37: // Left arrow
-                showPreviousImage();
-                break;
-            case 39: // Right arrow
-                showNextImage();
-                break;
-            case 27: // Esc
-                hideOverlay();
-                break;
+        case 37: // Left arrow
+            showPreviousImage();
+            break;
+        case 39: // Right arrow
+            showNextImage();
+            break;
+        case 27: // Esc
+            hideOverlay();
+            break;
         }
     }
 
@@ -377,7 +377,9 @@
         // Set overlay color
         try {
             overlay.style.backgroundColor = options.overlayBackgroundColor;
-        } catch (e) {}
+        } catch (e) {
+            // Silence the error and continue
+        }
     }
 
     function showOverlay(chosenImageIndex) {
@@ -705,5 +707,4 @@
         showNext: showNextImage,
         showPrevious: showPreviousImage
     };
-
 }));