Jarring message animation when opening keyboard
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 42/100
Research direction
Reproduce the issue from the chat entry point on an iPhone 13 running iOS 18.2.1, using a conversation with more than one screen of messages. Trace the message-list animation behavior while opening and closing the keyboard; done means both on-screen and off-screen messages slide together without disappearing, fading in, or overlapping.
Written by the indexing model from the issue text.
Description
- I have searched open and closed issues for duplicates
- I saw #5468 and #5129 which seemed related but different.
- I am submitting a bug report for existing functionality that does not work as intended
- This isn't a feature request or a discussion topic
Bug description
When opening/closing the keyboard in a chat, the animation for off-screen messages is different from the animation of on-screen messages and it is very jarring.
Steps to reproduce
- open a chat with more than a screen of messages
- open the keyboard (the animation is buggy)
- close the keyboard (the animation is buggy)
Actual result: Describe here what happens after you run the steps above (i.e. the buggy behaviour)
- When opening keyboard:
- The messages that are about to be off-screen just disappear.
- The on-screen messages (correctly) slide up.
- This isn't very noticeable, but is probably the same bug.
- When closing keyboard:
- The messages that were off-screen fade in.
- The on-screen messages (correctly) slide down.
- This is very jarring, as there is a period of time when messages overlap.
Expected result: Describe here what should happen after you run the steps above (i.e. what would be the correct behaviour)
Both on- and off- screen messages should slide up/down to animate.
Screenshots
Video:
https://github.com/user-attachments/assets/00802c91-9da1-4c9c-bcc9-6694fea63459
Showing keyboard:
Hiding keyboard:
Device info
Device: iPhone 13
iOS version: 18.2.1
Signal version: 7.43 (573)
Link to debug log
- Dominant language
- Swift
- Stars
- 12.2k
- Forks
- 3.5k
- PR merge metrics
- No merged PRs in 30d
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from signalapp/Signal-iOS
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
signalapp/Signal-iOS#6331 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 78/100
signalapp/Signal-iOS#6317 · 1 comment ·
-
In `splash view`, the ToS component must have also the `link` accessibility trait for Voice Over Open
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
signalapp/Signal-iOS#6309 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
signalapp/Signal-iOS#6296 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 52/100
signalapp/Signal-iOS#6351 ·
All issues in signalapp/Signal-iOS
Similar issues
-
tvOS
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
skiptools/skip-fuse-ui#147 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
OneBusAway/onebusaway-ios#1438 · 1 reaction ·