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

Pressing left arrow collapses list of emojis #39

Open
mbudde opened this issue Mar 11, 2024 · 2 comments
Open

Pressing left arrow collapses list of emojis #39

mbudde opened this issue Mar 11, 2024 · 2 comments
Labels
👆 good first issue Good for newcomers 👾 bug Something isn't working

Comments

@mbudde
Copy link

mbudde commented Mar 11, 2024

Bug Report

Current Behavior
Pressing (left arrow key) when focus is in the grid of emojis in a category (e.g. 'Food & Drink') causes the box to collapse. This makes it pretty much impossible to select an emoji using the keyboard only.

Reproduction steps:

  • Open emoji selector
  • Enter (select category)
  • four times to move focus to category box

Expected behavior/code
I would expect that pressing (left arrow key) would move focus to the emoji to the left like how moves focus to the right.

Environment

  • emoji-copy version(s): 16.0
  • OS: Arch Linux

Additional context/Screenshots
Screenshot from 2024-03-11 10-18-31
Screenshot from 2024-03-11 10-23-44

@FelipeFTN FelipeFTN added 👾 bug Something isn't working 👆 good first issue Good for newcomers labels Mar 11, 2024
@FelipeFTN
Copy link
Owner

Hello @mbudde! Nice catch! 💯
It seems this really is a bug 👾

Maybe there is some setting we missed on this component that is allowing it to collapse.
We shall take a look at the code, and find what is causing this!

Thank you so much for your report!
Also, if you want, feel free to explore the code and try finding ways to solve this bug! 💯

@mbudde
Copy link
Author

mbudde commented Mar 15, 2024

It looks like the issue comes from this piece of code:
https://gitlab.gnome.org/GNOME/gnome-shell/-/blob/881ec91370aab0b86f05845edc689534f9ba1ad6/js/ui/popupMenu.js#L1174-1178

I'm not sure how to prevent that from running. It might need to be fixed upstream.

@FelipeFTN FelipeFTN moved this to Not Started in Emoji Copy Mar 19, 2024
@FelipeFTN FelipeFTN moved this from Not Started to Planning in Emoji Copy Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👆 good first issue Good for newcomers 👾 bug Something isn't working
Projects
Status: Planning
Development

No branches or pull requests

2 participants