fix: only avoid scrollbar motion for prefers-reduced-motion (#1750)

After thinking about it, I do not believe the scrollbar is that distracting. But for prefers-reduced-motion we should be careful about the scrollbar growing so quickly.
This commit is contained in:
Nolan Lawson 2020-04-28 17:48:25 -07:00 committed by GitHub
parent e1532ed9d1
commit 5f6c5d89d1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 8 deletions

View File

@ -12,6 +12,7 @@
import { scheduleIdleTask } from '../../_utils/scheduleIdleTask' import { scheduleIdleTask } from '../../_utils/scheduleIdleTask'
import { createPriorityQueue } from '../../_utils/createPriorityQueue' import { createPriorityQueue } from '../../_utils/createPriorityQueue'
import { isMobile } from '../../_utils/userAgent/isMobile' import { isMobile } from '../../_utils/userAgent/isMobile'
import { store } from '../../_store/store'
// In Svelte's implementation of lists, these VirtualListLazyItems can be // In Svelte's implementation of lists, these VirtualListLazyItems can be
// created in any order. By default in fact it seems to do it in reverse // created in any order. By default in fact it seems to do it in reverse
@ -25,6 +26,7 @@
export default { export default {
async oncreate () { async oncreate () {
const { makeProps, key, index } = this.get() const { makeProps, key, index } = this.get()
const { reduceMotion } = this.store.get()
if (makeProps) { if (makeProps) {
await priorityQueue(index) await priorityQueue(index)
const props = await makeProps(key) const props = await makeProps(key)
@ -33,23 +35,22 @@
this.set({ props: props }) this.set({ props: props })
stop('VirtualListLazyItem set props') stop('VirtualListLazyItem set props')
} }
// On mobile, render in rIC for maximum input responsiveness. The reason we do // On desktop, if prefers-reduced-motion is enabled, avoid using scheduleIdleTask
// different behavior for desktop versus mobile is: // here because it causes the scrollbar to grow in a way that may sicken
// 1. On desktop, the scrollbar is really distracting as it changes size. It may // people with vestibular disorders.
// even cause issues for people with vestibular disorders (see also prefers-reduced-motion).
// 2. On mobile, the CPU is generally slower, so we want better input responsiveness.
// TODO: someday we can use isInputPending as a better way to break up work // TODO: someday we can use isInputPending as a better way to break up work
// https://www.chromestatus.com/feature/5719830432841728 // https://www.chromestatus.com/feature/5719830432841728
if (isMobile()) { if (!isMobile() && reduceMotion) {
scheduleIdleTask(setProps)
} else {
setProps() setProps()
} else {
scheduleIdleTask(setProps)
} }
} }
}, },
data: () => ({ data: () => ({
props: undefined props: undefined
}), }),
store: () => store,
components: { components: {
VirtualListItem VirtualListItem
} }