From 0f8296d384bf8cf0bf6674483c30ed3f3513c0a2 Mon Sep 17 00:00:00 2001
From: Olmo Kramer <olmo.kramer@gmail.com>
Date: Thu, 22 Mar 2018 23:45:16 +0100
Subject: [PATCH 1/8] Add hints.selectors setting

---
 doc/help/settings.asciidoc                   | 68 ++++++++++++++++++++
 qutebrowser/browser/hints.py                 | 22 +++++--
 qutebrowser/browser/navigate.py              |  5 +-
 qutebrowser/browser/webelem.py               | 27 +-------
 qutebrowser/config/configdata.yml            | 54 ++++++++++++++++
 tests/unit/browser/webkit/test_webkitelem.py | 68 +++++++++-----------
 6 files changed, 174 insertions(+), 70 deletions(-)

diff --git a/doc/help/settings.asciidoc b/doc/help/settings.asciidoc
index 75e684ffc..b545282ea 100644
--- a/doc/help/settings.asciidoc
+++ b/doc/help/settings.asciidoc
@@ -203,6 +203,7 @@
 |<<hints.next_regexes,hints.next_regexes>>|Comma-separated list of regular expressions to use for 'next' links.
 |<<hints.prev_regexes,hints.prev_regexes>>|Comma-separated list of regular expressions to use for 'prev' links.
 |<<hints.scatter,hints.scatter>>|Scatter hint key chains (like Vimium) or not (like dwb).
+|<<hints.selectors,hints.selectors>>|CSS selectors used to determine which elements on a page should have hints.
 |<<hints.uppercase,hints.uppercase>>|Make characters in hint strings uppercase.
 |<<history_gap_interval,history_gap_interval>>|Maximum time (in minutes) between two history items for them to be considered being from the same browsing session.
 |<<input.escape_quits_reporter,input.escape_quits_reporter>>|Allow Escape to quit the crash reporter.
@@ -2479,6 +2480,73 @@ Type: <<types,Bool>>
 
 Default: +pass:[true]+
 
+[[hints.selectors]]
+=== hints.selectors
+CSS selectors used to determine which elements on a page should have hints.
+
+This setting supports URL patterns.
+
+Type: <<types,Dict>>
+
+Default: 
+
+- +pass:[all]+:
+
+* +pass:[a]+
+* +pass:[area]+
+* +pass:[textarea]+
+* +pass:[select]+
+* +pass:[input:not([type=&quot;hidden&quot;])]+
+* +pass:[button]+
+* +pass:[frame]+
+* +pass:[iframe]+
+* +pass:[img]+
+* +pass:[link]+
+* +pass:[summary]+
+* +pass:[[onclick]]+
+* +pass:[[onmousedown]]+
+* +pass:[[role=&quot;link&quot;]]+
+* +pass:[[role=&quot;option&quot;]]+
+* +pass:[[role=&quot;button&quot;]]+
+* +pass:[[ng-click]]+
+* +pass:[[ngClick]]+
+* +pass:[[data-ng-click]]+
+* +pass:[[x-ng-click]]+
+- +pass:[images]+:
+
+* +pass:[img]+
+- +pass:[inputs]+:
+
+* +pass:[input[type=&quot;text&quot;]]+
+* +pass:[input[type=&quot;date&quot;]]+
+* +pass:[input[type=&quot;datetime-local&quot;]]+
+* +pass:[input[type=&quot;email&quot;]]+
+* +pass:[input[type=&quot;month&quot;]]+
+* +pass:[input[type=&quot;number&quot;]]+
+* +pass:[input[type=&quot;password&quot;]]+
+* +pass:[input[type=&quot;search&quot;]]+
+* +pass:[input[type=&quot;tel&quot;]]+
+* +pass:[input[type=&quot;time&quot;]]+
+* +pass:[input[type=&quot;url&quot;]]+
+* +pass:[input[type=&quot;week&quot;]]+
+* +pass:[input:not([type])]+
+* +pass:[textarea]+
+- +pass:[links]+:
+
+* +pass:[a[href]]+
+* +pass:[area[href]]+
+* +pass:[link[href]]+
+* +pass:[[role=&quot;link&quot;][href]]+
+- +pass:[media]+:
+
+* +pass:[audio]+
+* +pass:[img]+
+* +pass:[video]+
+- +pass:[urls]+:
+
+* +pass:[[src]]+
+* +pass:[[href]]+
+
 [[hints.uppercase]]
 === hints.uppercase
 Make characters in hint strings uppercase.
diff --git a/qutebrowser/browser/hints.py b/qutebrowser/browser/hints.py
index 6e5f4d7f4..27d869023 100644
--- a/qutebrowser/browser/hints.py
+++ b/qutebrowser/browser/hints.py
@@ -637,9 +637,8 @@ class HintManager(QObject):
                        star_args_optional=True, maxsplit=2)
     @cmdutils.argument('win_id', win_id=True)
     def start(self,  # pylint: disable=keyword-arg-before-vararg
-              group=webelem.Group.all, target=Target.normal,
-              *args, win_id, mode=None, add_history=False, rapid=False,
-              first=False):
+              group='all', target=Target.normal, *args, win_id, mode=None,
+              add_history=False, rapid=False, first=False):
         """Start hinting.
 
         Args:
@@ -741,10 +740,25 @@ class HintManager(QObject):
             raise cmdexc.CommandError("No URL set for this page yet!")
         self._context.args = args
         self._context.group = group
-        selector = webelem.SELECTORS[self._context.group]
+        selector = self._get_selector()
         self._context.tab.elements.find_css(selector, self._start_cb,
                                             only_visible=True)
 
+    def _get_selector(self):
+        """Get the CSS selectors for this url and hinting group."""
+        url = self._context.baseurl
+        group = self._context.group
+
+        selectors = config.instance.get('hints.selectors', url)
+        if group not in selectors:
+            selectors = config.val.hints.selectors
+
+            if group not in selectors:
+                raise cmdexc.CommandError("Undefined hinting group "
+                                          "'{}'!".format(group))
+
+        return ','.join(selectors[group])
+
     def current_mode(self):
         """Return the currently active hinting mode (or None otherwise)."""
         if self._context is None:
diff --git a/qutebrowser/browser/navigate.py b/qutebrowser/browser/navigate.py
index 257ce6fe0..dbe2b5ed9 100644
--- a/qutebrowser/browser/navigate.py
+++ b/qutebrowser/browser/navigate.py
@@ -21,7 +21,6 @@
 
 import posixpath
 
-from qutebrowser.browser import webelem
 from qutebrowser.config import config
 from qutebrowser.utils import objreg, urlutils, log, message, qtutils
 from qutebrowser.mainwindow import mainwindow
@@ -147,5 +146,5 @@ def prevnext(*, browsertab, win_id, baseurl, prev=False,
         else:
             browsertab.openurl(url)
 
-    browsertab.elements.find_css(webelem.SELECTORS[webelem.Group.links],
-                                 _prevnext_cb)
+    link_selector = 'a[href], area[href], link[href], [role=link][href]'
+    browsertab.elements.find_css(link_selector, _prevnext_cb)
diff --git a/qutebrowser/browser/webelem.py b/qutebrowser/browser/webelem.py
index 1d719738b..91bfd4d6e 100644
--- a/qutebrowser/browser/webelem.py
+++ b/qutebrowser/browser/webelem.py
@@ -17,14 +17,8 @@
 # You should have received a copy of the GNU General Public License
 # along with qutebrowser.  If not, see <http://www.gnu.org/licenses/>.
 
-"""Generic web element related code.
+"""Generic web element related code."""
 
-Module attributes:
-    Group: Enum for different kinds of groups.
-    SELECTORS: CSS selectors for different groups of elements.
-"""
-
-import enum
 import collections.abc
 
 from PyQt5.QtCore import QUrl, Qt, QEvent, QTimer
@@ -36,25 +30,6 @@ from qutebrowser.mainwindow import mainwindow
 from qutebrowser.utils import log, usertypes, utils, qtutils, objreg
 
 
-Group = enum.Enum('Group', ['all', 'links', 'images', 'url', 'inputs'])
-
-
-SELECTORS = {
-    Group.all: ('a, area, textarea, select, input:not([type=hidden]), button, '
-                'frame, iframe, link, summary, [onclick], [onmousedown], '
-                '[role=link], [role=option], [role=button], img, '
-                # Angular 1 selectors
-                '[ng-click], [ngClick], [data-ng-click], [x-ng-click]'),
-    Group.links: 'a[href], area[href], link[href], [role=link][href]',
-    Group.images: 'img',
-    Group.url: '[src], [href]',
-    Group.inputs: ('input[type=text], input[type=email], input[type=url], '
-                   'input[type=tel], input[type=number], '
-                   'input[type=password], input[type=search], '
-                   'input:not([type]), textarea'),
-}
-
-
 class Error(Exception):
 
     """Base class for WebElement errors."""
diff --git a/qutebrowser/config/configdata.yml b/qutebrowser/config/configdata.yml
index 2698c34b1..73184be82 100644
--- a/qutebrowser/config/configdata.yml
+++ b/qutebrowser/config/configdata.yml
@@ -1037,6 +1037,60 @@ hints.scatter:
 
     Ignored for number hints.
 
+hints.selectors:
+  default:
+    all:
+      - 'a'
+      - 'area'
+      - 'textarea'
+      - 'select'
+      - 'input:not([type="hidden"])'
+      - 'button'
+      - 'frame'
+      - 'iframe'
+      - 'img'
+      - 'link'
+      - 'summary'
+      - '[onclick]'
+      - '[onmousedown]'
+      - '[role="link"]'
+      - '[role="option"]'
+      - '[role="button"]'
+      - '[ng-click]'
+      - '[ngClick]'
+      - '[data-ng-click]'
+      - '[x-ng-click]'
+    links:
+      - 'a[href]'
+      - 'area[href]'
+      - 'link[href]'
+      - '[role="link"][href]'
+    images:
+      - 'img'
+    urls:
+      - '[src]'
+      - '[href]'
+    inputs:
+      - 'input[type="text"]'
+      - 'input[type="email"]'
+      - 'input[type="url"]'
+      - 'input[type="tel"]'
+      - 'input[type="number"]'
+      - 'input[type="password"]'
+      - 'input[type="search"]'
+      - 'input:not([type])'
+      - 'textarea'
+  type:
+    name: Dict
+    keytype: String
+    valtype:
+      name: List
+      none_ok: true
+      valtype: String
+  supports_pattern: true
+  desc: CSS selectors used to determine which elements on a page should have
+    hints.
+
 hints.uppercase:
   default: false
   type: Bool
diff --git a/tests/unit/browser/webkit/test_webkitelem.py b/tests/unit/browser/webkit/test_webkitelem.py
index df3de6310..eb06a3bca 100644
--- a/tests/unit/browser/webkit/test_webkitelem.py
+++ b/tests/unit/browser/webkit/test_webkitelem.py
@@ -29,7 +29,7 @@ import pytest
 from PyQt5.QtCore import QRect, QPoint, QUrl
 QWebElement = pytest.importorskip('PyQt5.QtWebKit').QWebElement
 
-from qutebrowser.browser import webelem, browsertab
+from qutebrowser.browser import browsertab
 from qutebrowser.browser.webkit import webkitelem
 from qutebrowser.misc import objects
 from qutebrowser.utils import usertypes
@@ -146,53 +146,46 @@ class SelectionAndFilterTests:
     TESTS = [
         ('<foo />', []),
         ('<foo bar="baz"/>', []),
-        ('<foo href="baz"/>', [webelem.Group.url]),
-        ('<foo src="baz"/>', [webelem.Group.url]),
+        ('<foo href="baz"/>', ['urls']),
+        ('<foo src="baz"/>', ['urls']),
 
-        ('<a />', [webelem.Group.all]),
-        ('<a href="foo" />', [webelem.Group.all, webelem.Group.links,
-                              webelem.Group.url]),
-        ('<a href="javascript://foo" />', [webelem.Group.all,
-                                           webelem.Group.links,
-                                           webelem.Group.url]),
+        ('<a />', ['all']),
+        ('<a href="foo" />', ['all', 'links', 'urls']),
+        ('<a href="javascript://foo" />', ['all', 'links', 'urls']),
 
-        ('<area />', [webelem.Group.all]),
-        ('<area href="foo" />', [webelem.Group.all, webelem.Group.links,
-                                 webelem.Group.url]),
+        ('<area />', ['all']),
+        ('<area href="foo" />', ['all', 'links', 'urls']),
 
-        ('<link />', [webelem.Group.all]),
-        ('<link href="foo" />', [webelem.Group.all, webelem.Group.links,
-                                 webelem.Group.url]),
+        ('<link />', ['all']),
+        ('<link href="foo" />', ['all', 'links', 'urls']),
 
-        ('<textarea />', [webelem.Group.all, webelem.Group.inputs]),
-        ('<select />', [webelem.Group.all]),
+        ('<textarea />', ['all', 'inputs']),
+        ('<select />', ['all']),
 
-        ('<input />', [webelem.Group.all, webelem.Group.inputs]),
+        ('<input />', ['all', 'inputs']),
         ('<input type="hidden" />', []),
-        ('<input type="text" />', [webelem.Group.inputs, webelem.Group.all]),
-        ('<input type="email" />', [webelem.Group.inputs, webelem.Group.all]),
-        ('<input type="url" />', [webelem.Group.inputs, webelem.Group.all]),
-        ('<input type="tel" />', [webelem.Group.inputs, webelem.Group.all]),
-        ('<input type="number" />', [webelem.Group.inputs, webelem.Group.all]),
-        ('<input type="password" />', [webelem.Group.inputs,
-                                       webelem.Group.all]),
-        ('<input type="search" />', [webelem.Group.inputs, webelem.Group.all]),
+        ('<input type="text" />', ['inputs', 'all']),
+        ('<input type="email" />', ['inputs', 'all']),
+        ('<input type="url" />', ['inputs', 'all']),
+        ('<input type="tel" />', ['inputs', 'all']),
+        ('<input type="number" />', ['inputs', 'all']),
+        ('<input type="password" />', ['inputs', 'all']),
+        ('<input type="search" />', ['inputs', 'all']),
 
-        ('<button />', [webelem.Group.all]),
-        ('<button href="foo" />', [webelem.Group.all, webelem.Group.url]),
+        ('<button />', ['all']),
+        ('<button href="foo" />', ['all', 'urls']),
 
         # We can't easily test <frame>/<iframe> as they vanish when setting
         # them via QWebFrame::setHtml...
 
-        ('<p onclick="foo" foo="bar"/>', [webelem.Group.all]),
-        ('<p onmousedown="foo" foo="bar"/>', [webelem.Group.all]),
-        ('<p role="option" foo="bar"/>', [webelem.Group.all]),
-        ('<p role="button" foo="bar"/>', [webelem.Group.all]),
-        ('<p role="button" href="bar"/>', [webelem.Group.all,
-                                           webelem.Group.url]),
+        ('<p onclick="foo" foo="bar"/>', ['all']),
+        ('<p onmousedown="foo" foo="bar"/>', ['all']),
+        ('<p role="option" foo="bar"/>', ['all']),
+        ('<p role="button" foo="bar"/>', ['all']),
+        ('<p role="button" href="bar"/>', ['all', 'urls']),
     ]
 
-    GROUPS = list(webelem.Group)
+    GROUPS = ['all', 'links', 'images', 'urls', 'inputs']
 
     COMBINATIONS = list(itertools.product(TESTS, GROUPS))
 
@@ -215,11 +208,12 @@ class TestSelectorsAndFilters:
         assert self.TESTS
 
     @pytest.mark.parametrize('group, val, matching', TESTS)
-    def test_selectors(self, webframe, group, val, matching):
+    def test_selectors(self, webframe, group, val, matching, config_stub):
         webframe.setHtml('<html><body>{}</body></html>'.format(val))
         # Make sure setting HTML succeeded and there's a new element
         assert len(webframe.findAllElements('*')) == 3
-        elems = webframe.findAllElements(webelem.SELECTORS[group])
+        selector = ','.join(config_stub.val.hints.selectors[group])
+        elems = webframe.findAllElements(selector)
         elems = [webkitelem.WebKitElement(e, tab=None) for e in elems]
         assert bool(elems) == matching
 

From 1d39276d24f71a37eaa2c32940fb17cc13ad61f7 Mon Sep 17 00:00:00 2001
From: Olmo Kramer <olmo.kramer@gmail.com>
Date: Sun, 22 Apr 2018 02:21:05 +0200
Subject: [PATCH 2/8] Add media hinting group

---
 qutebrowser/config/configdata.yml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/qutebrowser/config/configdata.yml b/qutebrowser/config/configdata.yml
index 73184be82..dbbb55855 100644
--- a/qutebrowser/config/configdata.yml
+++ b/qutebrowser/config/configdata.yml
@@ -1067,6 +1067,10 @@ hints.selectors:
       - '[role="link"][href]'
     images:
       - 'img'
+    media:
+      - 'audio'
+      - 'img'
+      - 'video'
     urls:
       - '[src]'
       - '[href]'

From 6212dbc488868b44e56ead49af50e08d82eb2a7d Mon Sep 17 00:00:00 2001
From: Olmo Kramer <olmo.kramer@gmail.com>
Date: Sun, 22 Apr 2018 02:21:23 +0200
Subject: [PATCH 3/8] Add more input types to the input hints group

---
 qutebrowser/config/configdata.yml | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/qutebrowser/config/configdata.yml b/qutebrowser/config/configdata.yml
index dbbb55855..d685f8b9a 100644
--- a/qutebrowser/config/configdata.yml
+++ b/qutebrowser/config/configdata.yml
@@ -1076,12 +1076,17 @@ hints.selectors:
       - '[href]'
     inputs:
       - 'input[type="text"]'
+      - 'input[type="date"]'
+      - 'input[type="datetime-local"]'
       - 'input[type="email"]'
-      - 'input[type="url"]'
-      - 'input[type="tel"]'
+      - 'input[type="month"]'
       - 'input[type="number"]'
       - 'input[type="password"]'
       - 'input[type="search"]'
+      - 'input[type="tel"]'
+      - 'input[type="time"]'
+      - 'input[type="url"]'
+      - 'input[type="week"]'
       - 'input:not([type])'
       - 'textarea'
   type:

From ca89b11398147a37e74574c21eeda2fab4a42c4f Mon Sep 17 00:00:00 2001
From: Olmo Kramer <olmo.kramer@gmail.com>
Date: Sun, 22 Apr 2018 03:24:34 +0200
Subject: [PATCH 4/8] Use configured links selector for navigate command

---
 qutebrowser/browser/navigate.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/qutebrowser/browser/navigate.py b/qutebrowser/browser/navigate.py
index dbe2b5ed9..f3da0c18e 100644
--- a/qutebrowser/browser/navigate.py
+++ b/qutebrowser/browser/navigate.py
@@ -146,5 +146,6 @@ def prevnext(*, browsertab, win_id, baseurl, prev=False,
         else:
             browsertab.openurl(url)
 
-    link_selector = 'a[href], area[href], link[href], [role=link][href]'
+    selectors = config.instance.get('hints.selectors', baseurl)
+    link_selector = ','.join(selectors['links'])
     browsertab.elements.find_css(link_selector, _prevnext_cb)

From 7aa5db29b5993260bde5f1b5dd309c2065808dc5 Mon Sep 17 00:00:00 2001
From: Olmo Kramer <olmo.kramer@gmail.com>
Date: Thu, 3 May 2018 23:23:12 +0200
Subject: [PATCH 5/8] Add tests for invalid hint group & js click handler

---
 .../end2end/data/hints/html/click_handler.html | 18 ++++++++++++++++++
 tests/end2end/features/hints.feature           |  5 +++++
 2 files changed, 23 insertions(+)
 create mode 100644 tests/end2end/data/hints/html/click_handler.html

diff --git a/tests/end2end/data/hints/html/click_handler.html b/tests/end2end/data/hints/html/click_handler.html
new file mode 100644
index 000000000..43338a519
--- /dev/null
+++ b/tests/end2end/data/hints/html/click_handler.html
@@ -0,0 +1,18 @@
+<!DOCTYPE html>
+
+<!-- target: hello.txt -->
+
+<html>
+    <head>
+        <meta charset="utf-8">
+        <title>Javascript link</title>
+    </head>
+    <body>
+        <button id="link">Follow me via JS!</button>
+        <script type="text/javascript">
+            document.getElementById('link').addEventListener('click', function() {
+                window.location.href = '/data/hello.txt';
+            });
+        </script>
+    </body>
+</html>
diff --git a/tests/end2end/features/hints.feature b/tests/end2end/features/hints.feature
index 3a0cb1da0..bb739b6bb 100644
--- a/tests/end2end/features/hints.feature
+++ b/tests/end2end/features/hints.feature
@@ -185,6 +185,11 @@ Feature: Using hints
         # The actual check is already done above
         Then no crash should happen
 
+    Scenario: Error with invalid hint group
+        When I open data/hints/buttons.html
+        And I run :hint INVALID_GROUP
+        Then the error "Undefined hinting group 'INVALID_GROUP'!" should be shown
+
     # https://github.com/qutebrowser/qutebrowser/issues/1613
     Scenario: Hinting inputs with padding
         When I open data/hints/input.html

From 332cbf811b1137231e85c1cb0368df3eb6df69ba Mon Sep 17 00:00:00 2001
From: Olmo Kramer <olmo.kramer@gmail.com>
Date: Fri, 4 May 2018 19:29:02 +0200
Subject: [PATCH 6/8] Add test for custom hint group

---
 tests/end2end/data/hints/custom_group.html | 13 +++++++++++++
 tests/end2end/features/hints.feature       | 11 +++++++++++
 2 files changed, 24 insertions(+)
 create mode 100644 tests/end2end/data/hints/custom_group.html

diff --git a/tests/end2end/data/hints/custom_group.html b/tests/end2end/data/hints/custom_group.html
new file mode 100644
index 000000000..d593b8231
--- /dev/null
+++ b/tests/end2end/data/hints/custom_group.html
@@ -0,0 +1,13 @@
+<!DOCTYPE html>
+
+<html>
+    <head>
+        <meta charset="utf-8">
+        <title>Custom hint groups</title>
+    </head>
+    <body>
+        <div class="clickable" onclick="console.log('beep!')">beep!</div>
+        <div class="clickable" onclick="console.log('bop!')">bop!</div>
+        <div class="clickable" onclick="console.log('boop!')">boop!</div>
+    </body>
+</html>
diff --git a/tests/end2end/features/hints.feature b/tests/end2end/features/hints.feature
index bb739b6bb..4372f732c 100644
--- a/tests/end2end/features/hints.feature
+++ b/tests/end2end/features/hints.feature
@@ -190,6 +190,17 @@ Feature: Using hints
         And I run :hint INVALID_GROUP
         Then the error "Undefined hinting group 'INVALID_GROUP'!" should be shown
 
+    Scenario: Custom hint group
+        When I open data/hints/custom_group.html
+        And I set hints.selectors to {"custom": [".clickable"]}
+        And I hint with args "--rapid custom"
+        And I run :follow-hint a
+        And I run :follow-hint s
+        And I run :follow-hint d
+        Then the javascript message "beep!" should be logged
+        And the javascript message "bop!" should be logged
+        And the javascript message "boop!" should be logged
+
     # https://github.com/qutebrowser/qutebrowser/issues/1613
     Scenario: Hinting inputs with padding
         When I open data/hints/input.html

From 9f5f52360c88fb65542e0f6923a3eddd6220ec45 Mon Sep 17 00:00:00 2001
From: Olmo Kramer <olmo.kramer@gmail.com>
Date: Wed, 4 Jul 2018 23:52:28 +0200
Subject: [PATCH 7/8] Add test cases for url-patterns

---
 tests/end2end/data/hints/custom_group.html |  2 --
 tests/end2end/features/hints.feature       | 22 +++++++++++++++-------
 2 files changed, 15 insertions(+), 9 deletions(-)

diff --git a/tests/end2end/data/hints/custom_group.html b/tests/end2end/data/hints/custom_group.html
index d593b8231..380d10dda 100644
--- a/tests/end2end/data/hints/custom_group.html
+++ b/tests/end2end/data/hints/custom_group.html
@@ -7,7 +7,5 @@
     </head>
     <body>
         <div class="clickable" onclick="console.log('beep!')">beep!</div>
-        <div class="clickable" onclick="console.log('bop!')">bop!</div>
-        <div class="clickable" onclick="console.log('boop!')">boop!</div>
     </body>
 </html>
diff --git a/tests/end2end/features/hints.feature b/tests/end2end/features/hints.feature
index 4372f732c..613101175 100644
--- a/tests/end2end/features/hints.feature
+++ b/tests/end2end/features/hints.feature
@@ -192,14 +192,22 @@ Feature: Using hints
 
     Scenario: Custom hint group
         When I open data/hints/custom_group.html
-        And I set hints.selectors to {"custom": [".clickable"]}
-        And I hint with args "--rapid custom"
-        And I run :follow-hint a
-        And I run :follow-hint s
-        And I run :follow-hint d
+        And I set hints.selectors to {"custom":[".clickable"]}
+        And I hint with args "custom" and follow a
+        Then the javascript message "beep!" should be logged
+
+    Scenario: Custom hint group with URL pattern
+        When I open data/hints/custom_group.html
+        And I run :set -u *://*/data/hints/custom_group.html hints.selectors '{"custom": [".clickable"]}'
+        And I hint with args "custom" and follow a
+        Then the javascript message "beep!" should be logged
+
+    Scenario: Fallback to global value with URL pattern set
+        When I open data/hints/custom_group.html
+        And I set hints.selectors to {"custom":[".clickable"]}
+        And I run :set -u *://*/data/hints/custom_group.html hints.selectors '{"other": [".other"]}'
+        And I hint with args "custom" and follow a
         Then the javascript message "beep!" should be logged
-        And the javascript message "bop!" should be logged
-        And the javascript message "boop!" should be logged
 
     # https://github.com/qutebrowser/qutebrowser/issues/1613
     Scenario: Hinting inputs with padding

From daa8bc05078e80991fcc1b05d9bfd44e345eb163 Mon Sep 17 00:00:00 2001
From: Olmo Kramer <olmo.kramer@gmail.com>
Date: Thu, 5 Jul 2018 00:57:08 +0200
Subject: [PATCH 8/8] Fix accidentally renamed hint group

---
 doc/help/settings.asciidoc                   |  2 +-
 qutebrowser/config/configdata.yml            |  2 +-
 tests/unit/browser/webkit/test_webkitelem.py | 18 +++++++++---------
 3 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/doc/help/settings.asciidoc b/doc/help/settings.asciidoc
index b545282ea..185dcb52d 100644
--- a/doc/help/settings.asciidoc
+++ b/doc/help/settings.asciidoc
@@ -2542,7 +2542,7 @@ Default:
 * +pass:[audio]+
 * +pass:[img]+
 * +pass:[video]+
-- +pass:[urls]+:
+- +pass:[url]+:
 
 * +pass:[[src]]+
 * +pass:[[href]]+
diff --git a/qutebrowser/config/configdata.yml b/qutebrowser/config/configdata.yml
index d685f8b9a..80d1fa68d 100644
--- a/qutebrowser/config/configdata.yml
+++ b/qutebrowser/config/configdata.yml
@@ -1071,7 +1071,7 @@ hints.selectors:
       - 'audio'
       - 'img'
       - 'video'
-    urls:
+    url:
       - '[src]'
       - '[href]'
     inputs:
diff --git a/tests/unit/browser/webkit/test_webkitelem.py b/tests/unit/browser/webkit/test_webkitelem.py
index eb06a3bca..ffbd63bae 100644
--- a/tests/unit/browser/webkit/test_webkitelem.py
+++ b/tests/unit/browser/webkit/test_webkitelem.py
@@ -146,18 +146,18 @@ class SelectionAndFilterTests:
     TESTS = [
         ('<foo />', []),
         ('<foo bar="baz"/>', []),
-        ('<foo href="baz"/>', ['urls']),
-        ('<foo src="baz"/>', ['urls']),
+        ('<foo href="baz"/>', ['url']),
+        ('<foo src="baz"/>', ['url']),
 
         ('<a />', ['all']),
-        ('<a href="foo" />', ['all', 'links', 'urls']),
-        ('<a href="javascript://foo" />', ['all', 'links', 'urls']),
+        ('<a href="foo" />', ['all', 'links', 'url']),
+        ('<a href="javascript://foo" />', ['all', 'links', 'url']),
 
         ('<area />', ['all']),
-        ('<area href="foo" />', ['all', 'links', 'urls']),
+        ('<area href="foo" />', ['all', 'links', 'url']),
 
         ('<link />', ['all']),
-        ('<link href="foo" />', ['all', 'links', 'urls']),
+        ('<link href="foo" />', ['all', 'links', 'url']),
 
         ('<textarea />', ['all', 'inputs']),
         ('<select />', ['all']),
@@ -173,7 +173,7 @@ class SelectionAndFilterTests:
         ('<input type="search" />', ['inputs', 'all']),
 
         ('<button />', ['all']),
-        ('<button href="foo" />', ['all', 'urls']),
+        ('<button href="foo" />', ['all', 'url']),
 
         # We can't easily test <frame>/<iframe> as they vanish when setting
         # them via QWebFrame::setHtml...
@@ -182,10 +182,10 @@ class SelectionAndFilterTests:
         ('<p onmousedown="foo" foo="bar"/>', ['all']),
         ('<p role="option" foo="bar"/>', ['all']),
         ('<p role="button" foo="bar"/>', ['all']),
-        ('<p role="button" href="bar"/>', ['all', 'urls']),
+        ('<p role="button" href="bar"/>', ['all', 'url']),
     ]
 
-    GROUPS = ['all', 'links', 'images', 'urls', 'inputs']
+    GROUPS = ['all', 'links', 'images', 'url', 'inputs']
 
     COMBINATIONS = list(itertools.product(TESTS, GROUPS))