Bug 162740 - AX: REGRESSION (200441): Controls fade out of a video makes the page scrolling back to video
Summary: AX: REGRESSION (200441): Controls fade out of a video makes the page scrollin...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: Safari 10
Hardware: Macintosh OS X 10.11
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2016-09-29 11:37 PDT by Aaron Chu
Modified: 2018-02-14 10:34 PST (History)
4 users (show)

See Also:


Attachments
Patch (1.56 KB, patch)
2016-09-29 11:48 PDT, Aaron Chu
no flags Details | Formatted Diff | Diff
Patch (879.97 KB, patch)
2016-09-30 13:31 PDT, Aaron Chu
beidson: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Aaron Chu 2016-09-29 11:37:05 PDT
When the controls of a video fades out then the page automatically scrolls back to the video
Comment 1 Aaron Chu 2016-09-29 11:48:25 PDT
Created attachment 290225 [details]
Patch
Comment 2 Sam Weinig 2016-09-29 15:07:55 PDT
Can this be tested?
Comment 3 Aaron Chu 2016-09-30 13:31:06 PDT
Created attachment 290374 [details]
Patch
Comment 4 chris fleizach 2016-10-20 17:42:37 PDT
Comment on attachment 290374 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=290374&action=review

> Source/WebCore/ChangeLog:10
> +        Removed the focus event that is fired on the invisible button which also causes the page to scroll.

this seems like we could add a layout test for this

after the page loads, we can confirm that the scroll offset is still zero
Comment 5 Brady Eidson 2018-02-14 10:34:14 PST
Comment on attachment 290374 [details]
Patch

Patches that have been up for review since 2016 are almost certainly too stale to be relevant to trunk in their current form.

If this patch is still important please rebase it and post it for review again.