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

Move keyboard handling to separate classes #1850

Merged
merged 3 commits into from
Dec 4, 2024

Conversation

CendioOssman
Copy link
Member

Encapsulate all the platform specific magic around keyboard in to specific classes, in order to keep the core code more readable.

We might not always be able to map to an RFB key code, so let's use the
platform specific code to track which keys are pressed.
Copy link
Member

@samhed samhed left a comment

Choose a reason for hiding this comment

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

Looks good! Nice to see viewport.cxx become a bit smaller..

Note that I didn't test it in action.

PS: Isn't the copyright year in the new files supposed to be -2024?

@CendioOssman
Copy link
Member Author

PS: Isn't the copyright year in the new files supposed to be -2024?

These patches have been sitting around for some time. :)

This is a general thing so move it in to the core library instead,
letting vncviewer focus on just translation of system events to VNC
ones.
Encapsulate all the platform specific magic around keyboard in to
specific classes, in order to keep the core code more readable.
@CendioOssman CendioOssman merged commit 994d2c6 into TigerVNC:master Dec 4, 2024
@CendioOssman CendioOssman deleted the keysplit branch December 4, 2024 16:12
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