diff options
| author | Phil Crosby | 2014-12-16 10:49:55 -0800 |
|---|---|---|
| committer | Phil Crosby | 2014-12-16 10:49:55 -0800 |
| commit | 46a5bad5d7d3532431dca3c567ad4f148f565e08 (patch) | |
| tree | 92e61825a45d470fd49440ef7cf4fec93233b4ce | |
| parent | 902a1f64fe5a22902b80f7e85bed91f8cdc0e57b (diff) | |
| download | vimium-46a5bad5d7d3532431dca3c567ad4f148f565e08.tar.bz2 | |
Add a little more context to a comment
| -rw-r--r-- | content_scripts/vimium_frontend.coffee | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/content_scripts/vimium_frontend.coffee b/content_scripts/vimium_frontend.coffee index 491f5750..fa20cd9e 100644 --- a/content_scripts/vimium_frontend.coffee +++ b/content_scripts/vimium_frontend.coffee @@ -227,8 +227,8 @@ setScrollPosition = (scrollX, scrollY) -> window.focusThisFrame = (shouldHighlight) -> if window.innerWidth < 3 or window.innerHeight < 3 # This frame is too small to focus. Cancel and tell the background frame to focus the next one instead. - # NOTE(smblott) We assume that there is at least one frame large enough to focus. - # See #1317. + # This affects sites like Google Inbox, which have many tiny iframes. See #1317. + # Here we're assuming that there is at least one frame large enough to focus. chrome.runtime.sendMessage({ handler: "nextFrame", frameId: frameId }) return window.focus() |