From 17d7b88ee05ee940404fef5bb605c9b7665b15b8 Mon Sep 17 00:00:00 2001 From: Vincent Bernat Date: Tue, 5 Aug 2014 09:11:37 +0200 Subject: Match mixed-case relationship links in goNext/goPrevious Link types are case-insensitive: http://www.w3.org/TR/html5/links.html#linkTypes This should fix #1115. --- content_scripts/vimium_frontend.coffee | 2 +- tests/dom_tests/dom_tests.coffee | 8 ++++++++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/content_scripts/vimium_frontend.coffee b/content_scripts/vimium_frontend.coffee index e45c4dfe..896253a6 100644 --- a/content_scripts/vimium_frontend.coffee +++ b/content_scripts/vimium_frontend.coffee @@ -811,7 +811,7 @@ findAndFollowRel = (value) -> for tag in relTags elements = document.getElementsByTagName(tag) for element in elements - if (element.hasAttribute("rel") && element.rel == value) + if (element.hasAttribute("rel") && element.rel.toLowerCase() == value) followLink(element) return true diff --git a/tests/dom_tests/dom_tests.coffee b/tests/dom_tests/dom_tests.coffee index 145b5866..0b2a9d89 100644 --- a/tests/dom_tests/dom_tests.coffee +++ b/tests/dom_tests/dom_tests.coffee @@ -230,6 +230,14 @@ context "Find prev / next links", goNext() assert.equal '#first', window.location.hash + should "match mixed case link relation", -> + document.getElementById("test-div").innerHTML = """ + + """ + goNext() + assert.equal '#first', window.location.hash + + createLinks = (n) -> for i in [0...n] by 1 link = document.createElement("a") -- cgit v1.2.3