Skip to content

Conversation

@firattale
Copy link
Contributor

Overview

What features/fixes does this PR include?
Adding animation after updating profile sections ( Availability, Timezone, Pronouns)
Please provide the GitHub issue number
#890

Closes #

Follow up Improvement Ideas

  • please list any improvement/ideas
  • We can this animation to other sections

Implementation

Describe technical (nontrivial / non-obvious) parts of your code
I created a React hook to keep animation logic separated from component and easily reusable.

Side effects

Assets

[Include screenshots/videos if it makes reviewing easier.]
https://gfycat.com/blindbiggroundbeetle

@vercel
Copy link

vercel bot commented Dec 9, 2021

@firattale is attempting to deploy a commit to the MetaFam Team on Vercel.

A member of the Team first needs to authorize it.

@vidvidvid
Copy link
Collaborator

nice, you pretty much got it right - but as you can see, when the value gets updated, the updated values flashes immediately, followed by fade out and fade in animation.
it would be nice if we made sure that the old value fades out and the new value fades in.

@firattale
Copy link
Contributor Author

Screenshot

The animation has updated.

@vidvidvid
Copy link
Collaborator

Great job! :)

@vidvidvid vidvidvid requested a review from alalonde December 10, 2021 16:00
@vidvidvid vidvidvid added the ready-for-review Add this label to your PR when its ready for review label Dec 10, 2021
Copy link
Member

@lucidcyborg lucidcyborg left a comment

Choose a reason for hiding this comment

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

LGTM

@lucidcyborg lucidcyborg merged commit d652bf6 into MetaFam:develop Dec 14, 2021
alalonde pushed a commit that referenced this pull request Dec 16, 2021
* add animation

* Change animation
alalonde added a commit that referenced this pull request Dec 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-review Add this label to your PR when its ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants