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

[Feature Request] Handle the tapback menu for messages at the top of the conversation #120

Closed
sabogalc opened this issue Apr 13, 2021 · 4 comments
Labels
wontfix This will not be worked on

Comments

@sabogalc
Copy link

I was inspired to look for this because of this ticket. Below is a video of how it behaves in SMServer.

SMServer.-.Brave.2021-04-13.14-38-14.mp4

Originally posted in open-imcore/imcore.react#27

This works well most of the time because of how SMServer pushes messages down when bringing up the tapback menu, but sometimes it's still unreachable.

@sabogalc
Copy link
Author

One thing I noticed is that on macOS (at least my version of Mojave), this bug behaves the same, but on iOS, it works pretty well.

Screen.Recording.2021-04-13.at.5.35.05.PM.mp4
RPReplay_Final1618349823.MP4

@itsjunetime
Copy link
Owner

Sorry, I'm somewhat confused — is it not possible to scroll up to view the tapback menu? Or were you just expecting that the tapback menu would show even when only part of the text was visible?

@sabogalc
Copy link
Author

Yes, the second part. I'm requesting that the tapback menu lowers the text enough to have the menu show up in use cases where the text is at the top. Or that it just goes through the top menu like it does on iOS. It is possible to scroll up, but sometimes this is just how the text naturally shows up when scrolled all the way down.

@itsjunetime
Copy link
Owner

Sorry, I simply don't think it's worth it to add something in to specifically scroll the main view down when you can easily do that manually. Closing this issue as a wontfix.

@itsjunetime itsjunetime added the wontfix This will not be worked on label Jun 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants