Add a semicolon at the end of this statement 55/94955/1
authoranushadasari <danush10@in.ibm.com>
Wed, 4 Sep 2019 16:51:55 +0000 (22:21 +0530)
committeranushadasari <danush10@in.ibm.com>
Wed, 4 Sep 2019 16:52:12 +0000 (22:22 +0530)
In JavaScript, the semicolon (;) is optional as a statement separator, but omitting semicolons can be confusing, and lead to unexpected results because a semicolon is implicitly inserted at the end of each line.

Issue-ID: VID-607
Change-Id: I92d5834ace5659334bdede1dfc2af23cc697f78a
Signed-off-by: anushadasari <danush10@in.ibm.com>
vid-app-common/src/main/webapp/app/vid/scripts/directives/popupWindowDirective.js

index 1ca3203..a26744d 100755 (executable)
@@ -64,7 +64,7 @@ var popupWindowDirective = function($log, $window) {
                scrollPosition = {\r
                    x : $window.pageXOffset,\r
                    y : $window.pageYOffset\r
-               }\r
+               };\r
                $window.scrollTo(0, 0);\r
                element.css("display", "table");\r
                element.prev().css("display", "block");\r
@@ -74,7 +74,7 @@ var popupWindowDirective = function($log, $window) {
                $window.scrollTo(scrollPosition.x, scrollPosition.y);\r
            }\r
        });\r
-    }\r
+    };\r
 \r
     return {\r
        restrict : "EA",\r
@@ -83,6 +83,6 @@ var popupWindowDirective = function($log, $window) {
        link : link,\r
        template : '<table style="display: none; position: absolute; left: 0; top: 0; width: 100%; height: 100%; border-collapse: collapse; margin: 0; padding: 0"> <tr><td align="center" style="vertical-align: top; padding: 10px"><div style="display: inline-block; padding: 5px; background-color: white" ng-transclude></div></td></tr></table>'\r
     };\r
-}\r
+};\r
 \r
 appDS2.directive("popupWindow", [ "$log", "$window", popupWindowDirective ]);\r