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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
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.