Tab Tray should respect "Reduce Motion" accessibility setting

NEW
Unassigned

Status

()

defect
4 years ago
a year ago

People

(Reporter: dusek, Unassigned)

Tracking

(Blocks 1 bug, {access})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

47 bytes, text/x-github-pull-request
dusek
: review+
Details | Review
Reporter

Description

4 years ago
"Reduce motion" is a setting to reduce animations, in my understanding especially those that can induce feeling of 3D motion and cause "not feeling well" state.

Tab Tray currently animates the tab in a sense that makes it feel "zoom out" into the Tab Tray, and then when selecting it, the selected tab web content "zooms in", thus this could create a perception of movement for the observing individual.

We should take a hint from App Switcher (which is in principle very similar to Tab Tray) to make e.g. a crossfade animation in/out of the Tab Tray instead of the "flying tabs" in case where the user has turned on the "General > Accessibility > Reduce Motion > Reduce Motion" setting.

Note that Safari does not do this with their tabs, so we would be better than Safari in that regard.

Comment 1

4 years ago
Posted file Pull request
Attachment #8634581 - Flags: review?(dusek)
Reporter

Comment 2

4 years ago
Comment on attachment 8634581 [details] [review]
Pull request

Thanks for your work! Comments in the PR.
Attachment #8634581 - Flags: review?(dusek) → review+
Reporter

Updated

4 years ago
Assignee: dusek → zhaocong89
Status: NEW → ASSIGNED
Duplicate of this bug: 1194374
Looks like Boris left some comments for review for arkratos to address. This code is also pretty stale since in the meantime there has been a lot of work done on the transition animation code. I can probably update this - doesn't seem to be a lot of code changes.
Assignee: zhaocong89 → nobody
Status: ASSIGNED → NEW

Comment 6

a year ago
Would be great if this were moved ahead with the other issues in VIMS. There's been movement in the W3C on this https://drafts.csswg.org/mediaqueries/
You need to log in before you can comment on or make changes to this bug.