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

Missing Xkb types #49

Closed
Ralith opened this issue Sep 30, 2016 · 6 comments
Closed

Missing Xkb types #49

Ralith opened this issue Sep 30, 2016 · 6 comments

Comments

@Ralith
Copy link

Ralith commented Sep 30, 2016

The various Xkb* events and flags don't seem to be bound. These are needed for reliable keyboard handling with xkbcommon-x11. It's easy to work around this by defining whatever you need locally and using transmute, but that leads to duplication of effort. The planned move to autogenerated bindings (#40) should fix this, but I thought I'd make a note anyway.

@ghost
Copy link

ghost commented Sep 30, 2016

I should be able to get on this tomorrow. Unfortunately, I'm waiting for untagged unions to become stable before moving to generated bindings.

@ghost
Copy link

ghost commented Oct 3, 2016

@Ralith 33be439 Are these the types and constants you were looking for?

@Ralith
Copy link
Author

Ralith commented Oct 3, 2016

Thanks! That's the types I needed. However, it looks like you missed the event type and event type mask constants (such as XkbMapNotify and XkbMapNotifyMask) starting around line 69 of X11/extensions/XKB.h (from kbproto).

@ghost
Copy link

ghost commented Oct 9, 2016

86d346b Sorry for the delayed response. I've been working two jobs, so my time for programming has been limited :( I'll publish these changes if that's everything you needed.

@Ralith
Copy link
Author

Ralith commented Oct 9, 2016

That looks like all the stuff I need, thanks!

No worries about the delay, if I couldn't wait I'd have made a PR. Fortunately Rust makes it pretty easy to bodge this stuff. Thanks for all your work!

@Ralith Ralith closed this as completed Oct 9, 2016
@ghost
Copy link

ghost commented Oct 10, 2016

Thanks and published!

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

No branches or pull requests

1 participant