Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

fix(drawer): Update motion to match spec #2398

Merged
merged 2 commits into from
Mar 19, 2018

Conversation

williamernest
Copy link
Contributor

No description provided.

@codecov-io
Copy link

codecov-io commented Mar 14, 2018

Codecov Report

Merging #2398 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2398   +/-   ##
=======================================
  Coverage   98.86%   98.86%           
=======================================
  Files         101      101           
  Lines        4151     4151           
  Branches      531      531           
=======================================
  Hits         4104     4104           
  Misses         47       47

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 670c349...dc8afd2. Read the comment docs.

Copy link
Contributor

@acdvorak acdvorak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Motion designer JN confirmed via email that we should indeed be using the "standard" easing curve for both enter and exit.

@williamernest williamernest merged commit 6417b51 into master Mar 19, 2018
@williamernest williamernest deleted the fix/drawer/update-animations branch March 19, 2018 21:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants