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

Focus locked to prompt boxes #114

Closed
1 task done
Myridium opened this issue Jan 26, 2023 · 3 comments · Fixed by #179
Closed
1 task done

Focus locked to prompt boxes #114

Myridium opened this issue Jan 26, 2023 · 3 comments · Fixed by #179
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@Myridium
Copy link

Running Latest Version

  • I am running the latest version

Processor

M1 (or later)

Intel processor

No response

Memory

16GB

What happened?

Focus remains on prompt boxes, even after clicking in the image gallery area. Arrow keys do not move between images. Cannot press spacebar to quickview, either.

Crash Log

No response

@Myridium Myridium added the bug Something isn't working label Jan 26, 2023
@godly-devotion
Copy link
Collaborator

See under Image menu for Gallery keyboard shortcuts.

@Myridium
Copy link
Author

Okay, good to know. However I think this is still poor user design. It is standard practice to bring focus into the area where the user clicks. Good UX practice should adhere to common conventions. After I click on an image, it puts a highlight border around that image. Selectable items are arranged in a grid, and the coloured highlight suggests that the window focus is on this grid. Therefore I expect to be able to use the arrow keys to navigate the items in that grid.

@godly-devotion
Copy link
Collaborator

I am open to pull request that fixes this behavior. I personally haven't figured out how to remove focus from the text input controls yet thus requiring the Cmd modifier key.

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

Successfully merging a pull request may close this issue.

2 participants