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

feat(bar): Keyboard Widget #1266

Merged
merged 8 commits into from
Feb 8, 2025
Merged

Conversation

Dethada
Copy link
Contributor

@Dethada Dethada commented Feb 7, 2025

Add new widget for komorebi bar to display the current keyboard input method.

Previously, current input method was retrieved via the Language::CurrentInputMethodLanguageTag()
call, which is inaccurate as it is retrieving the current input method of the bar itself, so it will
only be updated when the bar is in focus. The fix is getting the TID of the currently focused window
and read its input method.
@LGUG2Z
Copy link
Owner

LGUG2Z commented Feb 8, 2025

image

Code looks good to me (just needs a cargo +nightly fmt), but it looks like there is a padding issue with the icon.

Once you resolve these two things I think this should be good to merge! Great work!

@Dethada
Copy link
Contributor Author

Dethada commented Feb 8, 2025

Should all be fixed now

@LGUG2Z LGUG2Z merged commit 0bb2991 into LGUG2Z:master Feb 8, 2025
2 checks passed
LGUG2Z pushed a commit that referenced this pull request Feb 8, 2025
This commit is a squashed commit containing the below commits from
PR #1266, which introduces a new "Keyboard" widget, which is used to
display information about the user's currently selected keyboard input
language. This new widget has a data refresh interval of 1 second if not
specified by the user.

721d2ef
58373cd
ce27a76
fb9054a
55cc2fd
461a738
781b8d0
fa6bf6f
@LGUG2Z
Copy link
Owner

LGUG2Z commented Feb 8, 2025

This is has been merged and will go into the next release, congratulations on shipping your first feature in this project and thank you for your contribution! ♥️

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