Add enter animation to new timeline events - #7686
Conversation
Add the `WithEnterAnimation` wrapper to add an slide-up + fade in animation to timeline events with SYNC or LOCAL origin so new items either coming from sync or sent by the current users animate.
|
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #7686 +/- ##
===========================================
- Coverage 80.91% 80.90% -0.02%
===========================================
Files 2780 2780
Lines 81899 81944 +45
Branches 11284 11289 +5
===========================================
+ Hits 66265 66293 +28
- Misses 11290 11299 +9
- Partials 4344 4352 +8 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
bxdxnn
left a comment
There was a problem hiding this comment.
Funny, I've also started working on it recently and was able to get a smooth animation for the entire timeline instead of just a fade-in for the new message. It required some refactoring instead of just putting the row in a box, could you please take a look?
#7688


Content
Add the
WithEnterAnimationwrapper to add an slide-up + fade in animation to timeline events with SYNC or LOCAL origin so new items either coming from sync or sent by the current users animate when added to the timeline. Other items will work the same.Motivation and context
Implement #6966.
FWIW, I tried a more thorough implementation with
Modifier.animateItem()but there were lots of weird animations, as we had before:So I decided to drop that implementation and do something simpler.
Screenshots / GIFs
Screen_recording_20260910_145639.mp4
Tests
Test sending and receiving new events in the timeline, as well as redacting them or paginating.
Tested devices
Checklist