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(linux): mcompile query data 🐘 #8840

Merged
merged 16 commits into from
Jun 12, 2023

Conversation

SabineSIL
Copy link
Contributor

This is the first step necessary for PR #8839. The task is to query the base keyboard data of keyboards

connected to #8839
@keymanapp-test-bot skip

@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented May 22, 2023

User Test Results

Test specification and instructions

User tests are not required

Test Artifacts

@mcdurdin mcdurdin force-pushed the feature/linux-mcompile-query-data branch from b532101 to fedbfbf Compare May 24, 2023 07:46
@mcdurdin mcdurdin changed the title Feature/linux mcompile query data 🐘 feat(linux): mcompile query data 🐘 May 24, 2023
@mcdurdin mcdurdin added the feat label May 24, 2023
@mcdurdin mcdurdin modified the milestones: Future, A17S13, A17S14 May 24, 2023
@SabineSIL SabineSIL changed the base branch from master to feature/linux-mcompile May 31, 2023 08:22
Copy link
Contributor

@ermshiperete ermshiperete left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that GdkKeymap is deprecated in GTK 4. Therefore I'd prefer if we wouldn't use it.

One possible solution that would work with X11 as well as Wayland would be something along the lines of the implementation of gdk_wayland_keymap_get_entries_for_keycode.

xkb_keymap_new_from_names returns a struct xkb_keymap pointer. xkb_keymap_key_get_syms_by_level then allows to get the keyvals for a given keycode.

linux/mcompile/keymap/keymap.cpp Show resolved Hide resolved
linux/mcompile/keymap/keymap.cpp Outdated Show resolved Hide resolved
linux/mcompile/keymap/keymap.cpp Outdated Show resolved Hide resolved
linux/mcompile/keymap/keymap.cpp Outdated Show resolved Hide resolved
@SabineSIL
Copy link
Contributor Author

opened 2 issues for remaining topics ( #8972 + #8973 )

@mcdurdin mcdurdin modified the milestones: A17S14, A17S15 Jun 10, 2023
@SabineSIL SabineSIL merged commit b9da9b0 into feature/linux-mcompile Jun 12, 2023
@SabineSIL SabineSIL deleted the feature/linux-mcompile-query-data branch June 12, 2023 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants