Remove selection state change animations
ClosedPublic

Authored by ngraham on Jul 7 2019, 12:59 PM.

Details

Summary

In general animations are nice, but animations for changed hover states are often
very distracting. Animating color and opacity changes can produce a visual
artifact whereby blue turns into gray. Furthermore, having the state be animated means
that moving the mouse rapidly over list items makes the highlight not follow the
mouse closely, which looks awkward.

This patch resolves those issues by removing the animation effects.

Test Plan

Before:

After:

Diff Detail

Repository
R255 Elisa
Branch
remove-selection-animations (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 13674
Build 13692: arc lint + arc unit
ngraham requested review of this revision.Jul 7 2019, 12:59 PM
ngraham created this revision.
ngraham edited the summary of this revision. (Show Details)Jul 7 2019, 1:00 PM
ngraham edited the test plan for this revision. (Show Details)
astippich accepted this revision.Jul 8 2019, 6:34 PM
astippich added a subscriber: astippich.

+1

This revision is now accepted and ready to land.Jul 8 2019, 6:34 PM
ngraham edited the summary of this revision. (Show Details)Jul 9 2019, 12:26 AM
This revision was automatically updated to reflect the committed changes.