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

Add default exception to events #15

Merged
merged 2 commits into from
Oct 24, 2021
Merged

Add default exception to events #15

merged 2 commits into from
Oct 24, 2021

Conversation

Timozen
Copy link
Contributor

@Timozen Timozen commented Oct 1, 2021

Add default exception for handling events such that we can at least display the error message.
This is dirty fix for solving this issue, as all exception are caught like this.
We should investigate which exceptions could actually occur, and only use them.
Others should be caught by the users themselves.

This change at least allows users to understand the error which could appear in their own usage of the function.

Add default exception for handling events such that we can at least display the error message.
This is dirty fix for solving this issue, as all exepction are caught like this.
We should investigate which exceptions could actually occur, and only use them.
Others should be caught by the users themselves.
This commit fixes if a modifier key is pressed without additional key
Eg. previous would create control-control combies.
Now only control is created
Added shift key support
Replace format call with f-string
QWheelEvent works now with modifier keys
@ParisNeo ParisNeo merged commit 496482e into ParisNeo:master Oct 24, 2021
@ParisNeo
Copy link
Owner

OK, that is cool.
Pull request accepted.

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