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

Fixed Shift combinations with different keyboard layouts #309

Merged
merged 4 commits into from
Aug 20, 2024

Conversation

najlkin
Copy link
Contributor

@najlkin najlkin commented Aug 14, 2024

This is a possible fix for #308 . Please test it!

@justinlaughlin
Copy link
Contributor

Something funky going on with test 11 mesh-explorer on mac 👀

image

@najlkin
Copy link
Contributor Author

najlkin commented Aug 14, 2024

What is that !? 🙃

@najlkin
Copy link
Contributor Author

najlkin commented Aug 14, 2024

Can you test it @justinlaughlin ? I do not have a Mac 🍏 ...

@justinlaughlin
Copy link
Contributor

Can you test it @justinlaughlin ? I do not have a Mac 🍏 ...

I don't have one either unfortunately 🏠🐧

@najlkin
Copy link
Contributor Author

najlkin commented Aug 14, 2024

Ok, I maybe know what happens, SdlWindow::signalKeyDown() generates also these key down events, but does not initialize the scan code, which did not matter before, but matters now 🤔 .

@najlkin
Copy link
Contributor Author

najlkin commented Aug 14, 2024

Hmm, some keys are translated strangely to scan codes, but still ASCII, so the code works 😉
Edit: Nope, some are translated to zero and processed in keyDownEvent() instead of textInputEvent(), but somehow luckily it works 🙃

@najlkin
Copy link
Contributor Author

najlkin commented Aug 15, 2024

Ok, still some keys are handled in keyDownEvent() instead of textInputEvent(), but it does not matter, I would say, as the called functions are identified by the key code, which is always correct from signalKeyDown().

@tzanio
Copy link
Member

tzanio commented Aug 15, 2024

I can confirm that the latest commit shows the mesh-explorer test correctly on my mac

Copy link
Member

@tzanio tzanio left a comment

Choose a reason for hiding this comment

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

I tested some more, and this appears to work as before with the standard layout on Mac

@justinlaughlin justinlaughlin self-requested a review August 20, 2024 22:37
@justinlaughlin
Copy link
Contributor

FWIW, tested mesh-explorer on WSL and it seems to be fine.

@najlkin
Copy link
Contributor Author

najlkin commented Aug 20, 2024

@justinlaughlin Do you have different layouts than the US?

@tzanio tzanio merged commit 975049b into master Aug 20, 2024
10 checks passed
@tzanio tzanio deleted the najlkin/fix-key-locale branch August 20, 2024 23:42
@tzanio tzanio mentioned this pull request Aug 20, 2024
12 tasks
@justinlaughlin
Copy link
Contributor

@justinlaughlin Do you have different layouts than the US?

Just USA 🦅

najlkin added a commit that referenced this pull request Sep 27, 2024
Updated for #307, #309  and #316 .
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants