diff options
| -rw-r--r-- | linkHints.js | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/linkHints.js b/linkHints.js index 58d6e979..45f54a9d 100644 --- a/linkHints.js +++ b/linkHints.js @@ -71,7 +71,7 @@ function buildLinkHints() { // highlight all the links on the page; we don't want some link hints to have more chars than others. var digitsNeeded = Math.ceil(logXOfBase(visibleElements.length, settings.linkHintCharacters.length)); var linkHintNumber = 0; - for (var i = 0; i < visibleElements.length; i++) { + for (var i = 0, count = visibleElements.length; i < count; i++) { hintMarkers.push(createMarkerFor(visibleElements[i], linkHintNumber, digitsNeeded)); linkHintNumber++; } @@ -104,7 +104,7 @@ function getVisibleClickableElements() { var visibleElements = []; // Find all visible clickable elements. - for (var i = 0; i < resultSet.snapshotLength; i++) { + for (var i = 0, count = resultSet.snapshotLength; i < count; i++) { var element = resultSet.snapshotItem(i); var clientRect = element.getClientRects()[0]; @@ -114,7 +114,7 @@ function getVisibleClickableElements() { // If the link has zero dimensions, it may be wrapping visible // but floated elements. Check for this. if (clientRect && (clientRect.width == 0 || clientRect.height == 0)) { - for (var j = 0; j < element.children.length; j++) { + for (var j = 0, childrenCount = element.children.length; j < childrenCount; j++) { if (window.getComputedStyle(element.children[j], null).getPropertyValue('float') != 'none') { var childClientRect = element.children[j].getClientRects()[0]; if (isVisible(element.children[j], childClientRect)) { @@ -248,8 +248,9 @@ function highlightLinkMatches(searchString) { if (linkMarker.getAttribute("hintString").indexOf(searchString) == 0) { if (linkMarker.style.display == "none") linkMarker.style.display = ""; - for (var j = 0; j < linkMarker.childNodes.length; j++) - linkMarker.childNodes[j].className = (j >= searchString.length) ? "" : "matchingCharacter"; + var childNodes = linkMarker.childNodes; + for (var j = 0, childNodesCount = childNodes.length; j < childNodesCount; j++) + childNodes[j].className = (j >= searchString.length) ? "" : "matchingCharacter"; linksMatched.push(linkMarker.clickableItem); } else { linkMarker.style.display = "none"; @@ -274,8 +275,7 @@ function numberToHintString(number, numHintDigits) { } while (number > 0); // Pad the hint string we're returning so that it matches numHintDigits. - var hintStringLength = hintString.length; - for (var i = 0; i < numHintDigits - hintStringLength; i++) + for (var i = 0; i < numHintDigits - hintString.length; i++) hintString.unshift(settings.linkHintCharacters[0]); return hintString.join(""); } |
