Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chat top bar overlaps and stucks midway #19639

Closed
3 tasks
Parveshdhull opened this issue Apr 15, 2024 · 5 comments
Closed
3 tasks

Chat top bar overlaps and stucks midway #19639

Parveshdhull opened this issue Apr 15, 2024 · 5 comments

Comments

@Parveshdhull
Copy link
Member

Parveshdhull commented Apr 15, 2024

Summary

  • 1. Chat animation gets stuck mid way while opening keyboard or opening chat with few messages
    CleanShot_2024-04-08_at_22 44 27

more details https://discord.com/channels/1210237582470807632/1217173160416382986/1227014363081085002

Maybe we can directly animate to final stage instead of interpolate with scroll. (Please discuss with design team)

  • 2. Chat avatar overlaps chat title
    we have different animations then design for avatar and that's what causing overlap.
    In design animation video, top bar stays at same position but in our case its moving down
output-2024-04-15_18.24.16.mp4
Screenshot
@Parveshdhull
Copy link
Member Author

Thank you @BalogunofAfrica for picking this issue 🙏

As this issue is tracking all the bugs with top-bar, I added another bug originally reported in #19677 (review).
Please feel free to break issue in smaller issues for fixing or skip any of sub issue to be later fixed by someone else.

@ilmotta
Copy link
Contributor

ilmotta commented Jun 24, 2024

Hey @BalogunofAfrica, just checking, are you working on this issue? If not, could you unassign yourself?

Somewhat related, in terms of priorities, we have now to be more judgmental about the value of animations to our users and ignore most animation issues, unless they are really getting in the way of users. Curious @J-Son89 @smohamedjavid does this paragraph more or less reflect how you are prioritizing animations in wallet screens?

This chat header animation in particular is adding unnecessary complexity since most chats in real life will have at least a handful of messages and then the user won't even see/care about the animation because they won't scroll to the top of the message list. If we face more issues with this header we can consider eliminating this animation entirely. I remember one of the designers was keen on this idea (just so we keep in mind this is a possibility, not saying we should do this now of course).

@BalogunofAfrica BalogunofAfrica removed their assignment Jun 25, 2024
@ilmotta
Copy link
Contributor

ilmotta commented Oct 3, 2024

@Parveshdhull, about issue 1 Chat animation gets stuck mid way while opening keyboard or opening chat with few messages, could you verify if the problem is still present after the recent improvements in the composer and simplifications from #21313?

Problems 2 & 3 are more about aesthetics, not really a functional bug, and for those we are trying to add the label ui-polish and avoid the label bug. If it wasn't for problem 1 I would remove the label bug from this issue. Overall it's better if we create separate issues for each problem.

Thank you!

@ilmotta ilmotta added the ui-polish Aesthetics; not a functional bug label Oct 3, 2024
@ilmotta ilmotta removed the bug label Oct 14, 2024
@ilmotta
Copy link
Contributor

ilmotta commented Oct 15, 2024

@Parveshdhull, could you re-check this issue to see if it's still relevant? The composer was rewritten, the Jump To button feature was disabled, and the chat has been simplified in #21313, all of which might have fixed this issue. Thank you

@Parveshdhull
Copy link
Member Author

Thank you @ilmotta for pinging me. We no longer have a dynamic top bar, so this issue is no longer relevant anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

4 participants