diff options
| author | mrmr1993 | 2015-09-07 00:53:36 +0100 |
|---|---|---|
| committer | mrmr1993 | 2015-09-07 00:53:36 +0100 |
| commit | ed2a46b22e9c2a03392127bb9a85f67b88d790cc (patch) | |
| tree | cfdf5614a9ca929b76d98f1afc26c2abeac1f1e8 /content_scripts/scroller.coffee | |
| parent | d425d5c542c0883c078c0818fb4b567c8f57e7ff (diff) | |
| download | vimium-ed2a46b22e9c2a03392127bb9a85f67b88d790cc.tar.bz2 | |
Traverse shadow DOMs when looking for scrollable elements
Diffstat (limited to 'content_scripts/scroller.coffee')
| -rw-r--r-- | content_scripts/scroller.coffee | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/content_scripts/scroller.coffee b/content_scripts/scroller.coffee index 271a7854..5f10ab65 100644 --- a/content_scripts/scroller.coffee +++ b/content_scripts/scroller.coffee @@ -81,7 +81,7 @@ doesScroll = (element, direction, amount, factor) -> findScrollableElement = (element, direction, amount, factor) -> while element != document.body and not (doesScroll(element, direction, amount, factor) and shouldScroll(element, direction)) - element = element.parentElement || document.body + element = (DomUtils.getContainingElement element) ? document.body element # On some pages, document.body is not scrollable. Here, we search the document for the largest visible @@ -220,7 +220,12 @@ Scroller = init: -> handlerStack.push _name: 'scroller/active-element' - DOMActivate: (event) -> handlerStack.alwaysContinueBubbling -> activatedElement = event.target + DOMActivate: (event) -> handlerStack.alwaysContinueBubbling -> + # If event.path is present, the true event taget (potentially inside a Shadow DOM inside + # event.target) can be found as its first element. + # NOTE(mrmr1993): event.path has been renamed to event.deepPath in the spec, but this change is not + # yet implemented by Chrome. + activatedElement = event.deepPath?[0] ? event.path?[0] ? event.target CoreScroller.init() # scroll the active element in :direction by :amount * :factor. |
