Re: [PATCH 3 of 3] hgweb: add missing semicolons to followlines.js

[Denis Laxalde]

The series looks good to me, thanks.

Anton Shestakov a écrit :

# HG changeset patch
# User Anton Shestakov <a...@dwimlabs.net>
# Date 1510312446 -28800
#      Fri Nov 10 19:14:06 2017 +0800
# Node ID 981d7ebd7dc74a88493f39982649a52ed09a199b
# Parent  95cb67784e3c0d05e24e387acda45fd1c8aa4653
hgweb: add missing semicolons to followlines.js

Minor stylistic issues caught by jshint.

diff --git a/mercurial/templates/static/followlines.js 
b/mercurial/templates/static/followlines.js
--- a/mercurial/templates/static/followlines.js
+++ b/mercurial/templates/static/followlines.js
@@ -38,7 +38,7 @@ document.addEventListener('DOMContentLoa
      // element
      var selectableElements = Array.prototype.filter.call(
          sourcelines.children,
-        function(x) { return x.tagName === selectableTag });
+        function(x) { return x.tagName === selectableTag; });

var btnTitleStart = 'start following lines history from here';

      var btnTitleEnd = 'terminate line block selection here';
@@ -62,7 +62,7 @@ document.addEventListener('DOMContentLoa
      }

// extend DOM with CSS class for selection highlight and action buttons

-    var followlinesButtons = []
+    var followlinesButtons = [];
      for (var i = 0; i < selectableElements.length; i++) {
          selectableElements[i].classList.add('followlines-select');
          var btn = createButton();
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel