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

fix(developer): Treat Right Shift as Shift in debugger #10919

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

mcdurdin
Copy link
Member

@mcdurdin mcdurdin commented Mar 4, 2024

Fixes #10915.

Note that a more systemic fix to keyman core will involve ensuring that the debug list is always properly terminated, but that does not need to happen in beta.

User Testing

Fixes #10915.

Note that a more systemic fix to keyman core will involve ensuring that
the debug list is always properly terminated, but that does not need to
happen in beta.
@mcdurdin mcdurdin requested a review from darcywong00 as a code owner March 4, 2024 08:51
@keymanapp-test-bot keymanapp-test-bot bot added has-user-test user-test-required User tests have not been completed labels Mar 4, 2024
@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented Mar 4, 2024

User Test Results

Test specification and instructions

  • TEST_DEBUG_MODIFIERS (PASSED): Tested with the attached PR build Keyman Developer Version 17.0.281-beta-test-10919 on Windows 10 OS and here is my observation: 1. Opened a custom-made project in the Keyman Developer. 2. Compiled it. 3. Opened the debugger window. 4. Enter some letters. 5. Pressed Left Shift Key times, then pressing the Right Shift key multiple times doesn't crash the keyboard or show any error message. Seems to be working as expected. (notes)

Test Artifacts

@keymanapp-test-bot keymanapp-test-bot bot added this to the B17S3 milestone Mar 4, 2024
@mcdurdin mcdurdin linked an issue Mar 4, 2024 that may be closed by this pull request
@bharanidharanj
Copy link

Test Results

  • TEST_DEBUG_MODIFIERS (PASSED): Tested with the attached PR build Keyman Developer Version 17.0.281-beta-test-10919 on Windows 10 OS and here is my observation: 1. Opened a custom-made project in the Keyman Developer. 2. Compiled it. 3. Opened the debugger window. 4. Enter some letters. 5. Pressed Left Shift Key times, then pressing the Right Shift key multiple times doesn't crash the keyboard or show any error message. Seems to be working as expected.

@keymanapp-test-bot keymanapp-test-bot bot removed the user-test-required User tests have not been completed label Mar 5, 2024
Copy link
Contributor

@darcywong00 darcywong00 left a comment

Choose a reason for hiding this comment

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

lgtm

@mcdurdin mcdurdin merged commit 5a0f500 into beta Mar 7, 2024
5 checks passed
@mcdurdin mcdurdin deleted the fix/developer/10915-right-shift-in-developer-crash branch March 7, 2024 11:13
@keyman-server
Copy link
Collaborator

Changes in this pull request will be available for download in Keyman version 17.0.284-beta

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug(developer): Core crash debugging sil_ipa keyboard
4 participants