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

Context menu can go offscreen when more than one option is present #425

Closed
DanilaFe opened this issue Jul 27, 2021 · 1 comment
Closed
Labels
bug Something isn't working

Comments

@DanilaFe
Copy link
Contributor

Describe the bug
Opening a message's context menu (for reacting or deleting) at the bottom of the screen places the context menu very close to the bottom. With one additional option, such as replying, the context menu can go offscreen, and be inaccessible. For the last message in a chat, this makes it impossible to reach the context menu, since there's no way to scroll down to put more of it into view.

To Reproduce
Steps to reproduce the behavior:

  1. Open a chat that takes up the entire screen and ends in one of your messages (to see the delete option)
  2. Open the context menu of the bottom message
  3. See that the context menu is partially off-screen

Expected behavior
Element web displays the context message "upwards" at the bottom and "downwards" at the top, thus preventing it from going out of bounds.

Screenshots
Screenshot 2021-07-27 at 15-31-03 https hydrogen element io
Current version of Hydrogen, delete button is accessible but approaching end of screen
Screenshot 2021-07-27 at 15-31-19 Screenshot
WIP replies version of Hydrogen, where the reply button pushes the delete button off-screen.

Desktop:

  • OS: Linux
  • Browser: Firefox
  • Version: 89.0.2
@bwindels bwindels added the bug Something isn't working label Aug 4, 2021
@bwindels
Copy link
Contributor

Fixed in #509

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants