Some code improvements suggested by linter

[fnpeditor.git] / modules / documentHistory / documentHistory.js

diff --git a/modules/documentHistory/documentHistory.js b/modules/documentHistory/documentHistory.js
index 9a01f16..2647ee3 100644 (file)
--- a/modules/documentHistory/documentHistory.js
+++ b/modules/documentHistory/documentHistory.js
@@ -13,7 +13,7 @@ return function(sandbox) {
     var dom = $(_.template(mainTemplateSrc)());\r
     var domNodes = {\r
         itemList: dom.find('.rng-module-documentHistory-itemsList'),\r
     var dom = $(_.template(mainTemplateSrc)());\r
     var domNodes = {\r
         itemList: dom.find('.rng-module-documentHistory-itemsList'),\r

-    }\r
+    };\r

     var itemViews = [];\r
     \r
     \r
     var itemViews = [];\r
     \r
     \r


@@ -30,6 +30,10 @@ return function(sandbox) {
         });\r
         dialog.show();\r
     });\r
         });\r
         dialog.show();\r
     });\r

+    \r
+    dom.find('.btn.display').click(function(e) {\r
+        sandbox.publish('displayVersion', {version: historyItems.getSelected()[0]});\r
+    });\r

         \r
     var addHistoryItem = function(item, options) {\r
         historyItems.add(item);\r
         \r
     var addHistoryItem = function(item, options) {\r
         historyItems.add(item);\r


@@ -39,18 +43,18 @@ return function(sandbox) {
         if(options.animate) {\r
             view.dom.hide().slideDown();\r
         }\r
         if(options.animate) {\r
             view.dom.hide().slideDown();\r
         }\r

-    }\r
+    };\r

     \r
     var toggleItemViews = function(toggle) {\r
         itemViews.forEach(function(view) {\r
             if(!historyItems.isSelected(view.item))\r
                 view.toggle(toggle);\r
         });\r
     \r
     var toggleItemViews = function(toggle) {\r
         itemViews.forEach(function(view) {\r
             if(!historyItems.isSelected(view.item))\r
                 view.toggle(toggle);\r
         });\r

-    }\r
+    };\r

     \r
     var toggleButton = function(btn, toggle) {\r
         dom.find('button.'+btn).toggleClass('disabled', !toggle);\r
     \r
     var toggleButton = function(btn, toggle) {\r
         dom.find('button.'+btn).toggleClass('disabled', !toggle);\r

-    }\r
+    };\r

     \r
     var historyItems = {\r
         _itemsById: {},\r
     \r
     var historyItems = {\r
         _itemsById: {},\r


@@ -80,18 +84,18 @@ return function(sandbox) {
             var len = this._selected.length;\r
             if(len === 0) {\r
                 toggleButton('compare', false);\r
             var len = this._selected.length;\r
             if(len === 0) {\r
                 toggleButton('compare', false);\r

-                toggleButton('show2', false);\r
+                toggleButton('display', false);\r

                 toggleButton('restore', false);\r
             }\r
             if(len === 1) {\r
                 toggleButton('compare', false);\r
                 toggleButton('restore', false);\r
             }\r
             if(len === 1) {\r
                 toggleButton('compare', false);\r

-                toggleButton('show2', true);\r
+                toggleButton('display', true);\r

                 toggleButton('restore', true);\r
             }\r
             if(len === 2) {\r
                 toggleItemViews(false);\r
                 toggleButton('compare', true);\r
                 toggleButton('restore', true);\r
             }\r
             if(len === 2) {\r
                 toggleItemViews(false);\r
                 toggleButton('compare', true);\r

-                toggleButton('show2', false);\r
+                toggleButton('display', false);\r

                 toggleButton('restore', false);\r
             } else {\r
                 toggleItemViews(true);\r
                 toggleButton('restore', false);\r
             } else {\r
                 toggleItemViews(true);\r


@@ -136,7 +140,7 @@ return function(sandbox) {
         getView: function() {\r
             return dom;\r
         }\r
         getView: function() {\r
             return dom;\r
         }\r

-    }\r
-}\r
+    };\r
+};\r

 \r
 });
\ No newline at end of file
 \r
 });
\ No newline at end of file