From df8c48a3b85d19add9f1f7d3962110a474380867 Mon Sep 17 00:00:00 2001
From: Simon Lydell <simon.lydell@gmail.com>
Date: Tue, 23 Feb 2016 07:23:39 +0100
Subject: [PATCH] Consider multimedia elements as adjustable

`<video>`, `<audio>`, `<embed>` and `<object>`. This allows pressing `<space>`
to toggle between play and pause, for instance.

This also allows to get rid of the check for fullscreen elements. That's good,
because _any_ element can be fullscreened, not just `<video>` elements.
---
 documentation/api.md       | 4 ++--
 extension/lib/utils.coffee | 1 +
 extension/lib/vimfx.coffee | 3 +--
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/documentation/api.md b/documentation/api.md
index 11dbdf5..7e89951 100644
--- a/documentation/api.md
+++ b/documentation/api.md
@@ -556,8 +556,8 @@ A `match` object has the following properties:
     key: see the [`activatable_element_keys`] option.
   - `'adjustable'`: element: an “adjustable” element (form control or video
     player). key: see the [`adjustable_element_keys`] option.
-  - `'other'`: element: some other kind of element that can receive keystrokes,
-    for example an element in fullscreen mode. key: any pressed key.
+  - `'other'`: element: some other kind of element that can receive keystrokes.
+    key: any pressed key.
 
   If none of the above criteria is met, the value is `null`, which means that
   the currently focused element does not appear to respond to keystrokes in any
diff --git a/extension/lib/utils.coffee b/extension/lib/utils.coffee
index 7aa72ff..e890358 100644
--- a/extension/lib/utils.coffee
+++ b/extension/lib/utils.coffee
@@ -64,6 +64,7 @@ isAdjustable = (element) ->
            'checkbox', 'radio', 'file', 'color'
            'date', 'time', 'datetime', 'datetime-local', 'month', 'week'
          ] or
+         element.localName in ['video', 'audio', 'embed', 'object'] or
          element instanceof XULControlElement or
          # Youtube special case.
          element.classList?.contains('html5-video-player') or
diff --git a/extension/lib/vimfx.coffee b/extension/lib/vimfx.coffee
index 1d1eb72..593d5f5 100644
--- a/extension/lib/vimfx.coffee
+++ b/extension/lib/vimfx.coffee
@@ -131,8 +131,7 @@ class VimFx extends EventEmitter
     document = event.originalTarget?.ownerDocument
     if focusType == null and document and
        # TODO: Remove when Tab Groups have been removed.
-       (vim.window.TabView?.isVisible() or
-        document.fullscreenElement or document.mozFullScreenElement)
+       vim.window.TabView?.isVisible()
       return 'other'
 
     keys = @options["#{focusType}_element_keys"]
-- 
2.39.3