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.

Sign up for GitHub

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

Jump to bottom

Rewrite bindings #109

Open
12 of 17 tasks
rdbende opened this issue Nov 3, 2022 · 9 comments · May be fixed by #132
Open
12 of 17 tasks

Rewrite bindings #109

rdbende opened this issue Nov 3, 2022 · 9 comments · May be fixed by #132
Labels
enhancement New feature or request
Milestone
0.3.0

Comments

@rdbende
Copy link
Member

rdbende commented Nov 3, 2022 •
edited

Implement binding API
Beta Give feedback

Event classes
Beta Give feedback

Helpers
Beta Give feedback

Tests
Beta Give feedback
@Moosems
Copy link
Collaborator

Moosems commented Nov 4, 2022

Why not?

@rdbende
Copy link
Member Author

rdbende commented Nov 4, 2022

Why not?

What?

@rdbende rdbende added the enhancement New feature or request label Nov 4, 2022
@Moosems
Copy link
Collaborator

Moosems commented Nov 4, 2022

Why can't you do keyboard modifiers on non-keyboard events?

@rdbende
Copy link
Member Author

rdbende commented Nov 4, 2022

Because there is a bug in the event type matching mechanism, and I haven't had time to fix it yet.

@rdbende rdbende added the help wanted Extra attention is needed label Dec 23, 2022
@rdbende rdbende changed the title Bindings still suck Rewrite bindings Apr 7, 2023
@rdbende
Copy link
Member Author

rdbende commented Apr 7, 2023

Ahh, these tasklists are cool.

@rdbende rdbende linked a pull request Apr 9, 2023 that will close this issue
Open
@Moosems
Copy link
Collaborator

Moosems commented Apr 9, 2023

Make sure to add .bind_class() in this, it can be incredibly useful.

@rdbende
Copy link
Member Author

rdbende commented Apr 9, 2023

It's already added! :))

@rdbende
Copy link
Member Author

rdbende commented Apr 9, 2023

You can call .bind() on a class instead of an instance.

@Moosems
Copy link
Collaborator

Moosems commented Apr 9, 2023

Rdbende, you sir, are epic.

@rdbende rdbende removed help wanted Extra attention is needed Priority: high labels Apr 9, 2023
@rdbende rdbende added this to the 0.3.0 milestone May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
0.3.0
Development

Successfully merging a pull request may close this issue.

Rewrite bindings tukaan/tukaan
2 participants
@rdbende @Moosems