Merge branch 'master' of ssh://cmpl.cc:2323/qutebrowser

qutebrowser/browser/commands.py

@@ -712,9 +712,11 @@ class CommandDispatcher:
         """
         frame = self._tabs.currentWidget().page().currentFrame()
         elem = frame.findFirstElement(webelem.SELECTORS[
-            webelem.Group.editable_focused])
+            webelem.Group.focus])
         if elem.isNull():
-            raise CommandError("No editable element focused!")
+            raise CommandError("No element focused!")
+        if not webelem.is_editable(elem):
+            raise CommandError("Focused element is not editable!")
         text = elem.evaluateJavaScript('this.value')
         self._editor = ExternalEditor(self._tabs)
         self._editor.editing_finished.connect(

qutebrowser/test/stubs.py

@@ -107,6 +107,10 @@ class FakeWebElement:
         self._attributes = attributes
         self._classes = classes
 
+    def toOuterXml(self):
+        """Imitate toOuterXml."""
+        return '<fakeelem>'
+
     def styleProperty(self, name, strategy):
         """Return the CSS style property named name.
 

qutebrowser/utils/webelem.py

@@ -36,7 +36,7 @@ import qutebrowser.config.config as config
 
 
 Group = enum('all', 'links', 'images', 'editable', 'url', 'prevnext_rel',
-             'prevnext', 'editable_focused')
+             'prevnext', 'focus')
 
 
 SELECTORS = {
@@ -45,19 +45,10 @@ SELECTORS = {
                 '[role=option], [role=button], img'),
     Group.links: 'a, area, link',
     Group.images: 'img',
-    # This doesn't contain all the groups where we should switch to insert mode
-    # - it is just used when opening the external editor.
-    Group.editable_focused: ('input[type=text]:focus, '
-                             'input[type=email]:focus, '
-                             'input[type=url]:focus, '
-                             'input[type=tel]:focus, '
-                             'input[type=number]:focus, '
-                             'input[type=password]:focus, '
-                             'input[type=search]:focus, '
-                             'textarea:focus'),
     Group.url: '[src], [href]',
     Group.prevnext_rel: 'a, area, link, [role=link]',
     Group.prevnext: 'a, area, button, [role=button]',
+    Group.focus: '*:focus',
 }
 
 FILTERS = {
@@ -208,6 +199,8 @@ def is_editable(elem):
         True if we should switch to insert mode, False otherwise.
     """
     # Beginnings of div-classes which are actually some kind of editor.
+    log.misc.debug("Checking if element is editable: {}".format(
+        elem.toOuterXml()))
     div_classes = ['CodeMirror',  # Javascript editor over a textarea
                    'kix-']        # Google Docs editor
     tag = elem.tagName().lower()