diff --git a/.eslintrc b/.eslintrc index 2eec935a7..ab23d427e 100644 --- a/.eslintrc +++ b/.eslintrc @@ -16,7 +16,7 @@ rules: no-throw-literal: 2 no-void: 2 radix: 2 - wrap-iife: 2 + wrap-iife: [2, "inside"] brace-style: [2, "1tbs", {"allowSingleLine": true}] comma-style: [2, "last"] consistent-this: [2, "self"] diff --git a/qutebrowser/javascript/position_caret.js b/qutebrowser/javascript/position_caret.js index 36bf0451f..62e610245 100644 --- a/qutebrowser/javascript/position_caret.js +++ b/qutebrowser/javascript/position_caret.js @@ -81,27 +81,28 @@ function isElementInViewport(node) { return boundingRect.top >= -20; } - -var walker = document.createTreeWalker(document.body, 4, null); -var node; -var textNodes = []; -while (node = walker.nextNode()) { - if (node.nodeType === 3 && node.data.trim() !== '') { - textNodes.push(node); +(function() { + var walker = document.createTreeWalker(document.body, 4, null); + var node; + var textNodes = []; + while (node = walker.nextNode()) { + if (node.nodeType === 3 && node.data.trim() !== '') { + textNodes.push(node); + } } -} -for (var i = 0; i < textNodes.length; i++) { - var element = textNodes[i].parentElement; - if (isElementInViewport(element.parentElement)) { - el = element; - break; + for (var i = 0; i < textNodes.length; i++) { + var element = textNodes[i].parentElement; + if (isElementInViewport(element.parentElement)) { + el = element; + break; + } } -} -if (el !== undefined) { - var range = document.createRange(); - range.setStart(el, 0); - range.setEnd(el, 0); - var sel = window.getSelection(); - sel.removeAllRanges(); - sel.addRange(range); -} + if (el !== undefined) { + var range = document.createRange(); + range.setStart(el, 0); + range.setEnd(el, 0); + var sel = window.getSelection(); + sel.removeAllRanges(); + sel.addRange(range); + } +})();