WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
186529
Disable some vestibular trigger animations when prefers-reduced-motion is active.
https://bugs.webkit.org/show_bug.cgi?id=186529
Summary
Disable some vestibular trigger animations when prefers-reduced-motion is act...
James Craig
Reported
2018-06-11 10:53:50 PDT
Disable some vestibular trigger animations when prefers-reduced-motion is active. For example, the blog post text floats upward right after a page load. Regression since the time that this post went live.
https://webkit.org/blog/7551/responsive-design-for-motion/
Attachments
Patch
(5.56 KB, patch)
2018-06-14 13:44 PDT
,
Jon Davis
no flags
Details
Formatted Diff
Diff
Patch
(1.52 KB, patch)
2018-07-13 13:53 PDT
,
Jon Davis
no flags
Details
Formatted Diff
Diff
Archive of layout-test-results from ews205 for win-future
(12.80 MB, application/zip)
2018-07-13 16:46 PDT
,
EWS Watchlist
no flags
Details
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Jon Davis
Comment 1
2018-06-14 13:44:13 PDT
Created
attachment 342758
[details]
Patch
James Craig
Comment 2
2018-07-13 13:14:30 PDT
Comment on
attachment 342758
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=342758&action=review
There are a bunch of whitespace-only diffs that should not be in this patch.
> Websites/webkit.org/wp-content/themes/webkit/style.css:1709 > + article,
Do you really want this on all article elements? Maybe this style sheet already has limited inclusion? I'm okay with this if you're sure, but I'd expect you'd want to limit the motion (and subsequent reduction) to the primary article on blog posts only: Looks like the article has a post classname. e.g. selector could be `article.post { ... }`
Jon Davis
Comment 3
2018-07-13 13:53:15 PDT
Created
attachment 344977
[details]
Patch
Jon Davis
Comment 4
2018-07-13 13:56:47 PDT
> Do you really want this on all article elements? Maybe this style sheet already has limited inclusion? I'm okay with this if you're sure, but I'd expect you'd want to limit the motion (and subsequent reduction) to the primary article on blog posts only: Looks like the article has a post classname. e.g. selector could be `article.post { ... }`
Yes, it should be on all article elements and the feature status page. This is a consistent page load animation across all content pages and posts on webkit.org. The motion should be replaced everywhere it is applied.
EWS Watchlist
Comment 5
2018-07-13 16:46:38 PDT
Comment on
attachment 344977
[details]
Patch
Attachment 344977
[details]
did not pass win-ews (win): Output:
https://webkit-queues.webkit.org/results/8530553
New failing tests: http/tests/security/canvas-remote-read-remote-video-redirect.html
EWS Watchlist
Comment 6
2018-07-13 16:46:49 PDT
Created
attachment 345003
[details]
Archive of layout-test-results from ews205 for win-future The attached test failures were seen while running run-webkit-tests on the win-ews. Bot: ews205 Port: win-future Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
WebKit Commit Bot
Comment 7
2018-07-24 10:31:25 PDT
Comment on
attachment 344977
[details]
Patch Clearing flags on attachment: 344977 Committed
r234155
: <
https://trac.webkit.org/changeset/234155
>
WebKit Commit Bot
Comment 8
2018-07-24 10:31:26 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 9
2018-07-24 10:33:27 PDT
<
rdar://problem/42546539
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug