Skip to content

Added enter key as a way to invoke the current focused FlowBoxChild c…#5

Open
Ano-sys wants to merge 1 commit into
pshycodr:mainfrom
Ano-sys:emoji_enter_selection
Open

Added enter key as a way to invoke the current focused FlowBoxChild c…#5
Ano-sys wants to merge 1 commit into
pshycodr:mainfrom
Ano-sys:emoji_enter_selection

Conversation

@Ano-sys

@Ano-sys Ano-sys commented Jun 11, 2026

Copy link
Copy Markdown

Add keyboard shortcuts for FlowBox navigation

Enter now activates the currently focused FlowBoxChild to copy the emoji to the clipboard.
The c key returns focus to the category bar while navigating through the FlowBoxes.

…licked event (copy emoji to clip) - added c key event to return to category bar when arrow navigating through FlowBox - window is terminated with exit(0) with method _close_window now, previously it was just hide
@pshycodr

Copy link
Copy Markdown
Owner

Hello @Ano-sys , thanks for the contribution!

I tested the changes and noticed a couple of things:

  • Previously, pressing Esc would just hide the OmniGlyph window and keep the app running in the background. With this change, Esc now closes the application completely via _close_window(), so the behavior is different from before.

  • The Enter key functionality is working as expected.

  • The c key doesn't seem to be working on my end. Pressing c doesn't open or focus the category bar.

Could you have a look?

@Ano-sys

Ano-sys commented Jun 12, 2026

Copy link
Copy Markdown
Author

Hi, I wrote a comment under the issue to explain my view on the escape behavior.

For c the focus is moved not to the main category bar with Emoji, Emoticons, Arrows, ..., instead the emoji category selection is focused. I fond myself stuck in the lower emoji selection, when navigating with the keyboard, this was a fast and easy solution to escape the capture int the FlowBox.

@pshycodr

Copy link
Copy Markdown
Owner

Thanks for clarifying.

I see what you mean now. The c shortcut is moving focus to the emoji category selector rather than the main collection sidebar. The Enter functionality works well, but I would like to think a bit more about the c shortcut and whether that's the best keybinding and navigation flow before merging.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants