Skip to content

fix(mac): ignore non-Unicode key caps when preparing OSK#15008

Merged
mcdurdin merged 1 commit intomasterfrom
fix/mac/15007-ignore-ansi-kvk
Oct 27, 2025
Merged

fix(mac): ignore non-Unicode key caps when preparing OSK#15008
mcdurdin merged 1 commit intomasterfrom
fix/mac/15007-ignore-ansi-kvk

Conversation

@mcdurdin
Copy link
Copy Markdown
Member

The typeFlags property is used to differentiate Unicode and non-Unicode keys in the array of keys in a .kvk file. All non-Unicode keys should be ignored, as we do not support non-Unicode keyboards on macOS.

The reason this fixes #15007, is that the keyboard in question had both ANSI and Unicode sections in its .kvk file, and the ANSI section was stored after the Unicode section, so the font spec for ANSI was overriding the previously (correctly) set Unicode font spec.

Fixes: #15007

User Testing

TEST_OSK_GALAXIE_HEBREW: Verify that the Galaxie Hebrew Mnemonic keyboard now displays in the OSK correctly on macOS

The typeFlags property is used to differentiate Unicode and non-Unicode
keys in the array of keys in a .kvk file. All non-Unicode keys should
be ignored, as we do not support non-Unicode keyboards on macOS.

The reason this fixes #15007, is that the keyboard in question had both
ANSI and Unicode sections in its .kvk file, and the ANSI section was
stored after the Unicode section, so the font spec for ANSI was
overriding the previously (correctly) set Unicode font spec.

Fixes: #15007
@keymanapp-test-bot
Copy link
Copy Markdown

keymanapp-test-bot bot commented Oct 24, 2025

User Test Results

Test specification and instructions

Test Artifacts

@keymanapp-test-bot keymanapp-test-bot bot added this to the A19S14 milestone Oct 24, 2025
mcdurdin added a commit that referenced this pull request Oct 24, 2025
The typeFlags property is used to differentiate Unicode and non-Unicode
keys in the array of keys in a .kvk file. All non-Unicode keys should
be ignored, as we do not support non-Unicode keyboards on macOS.

The reason this fixes #15007, is that the keyboard in question had both
ANSI and Unicode sections in its .kvk file, and the ANSI section was
stored after the Unicode section, so the font spec for ANSI was
overriding the previously (correctly) set Unicode font spec.

Fixes: #15007
Cherry-pick-of: #15008
Test-bot: skip
@sgschantz
Copy link
Copy Markdown
Contributor

Test Results

  • TEST_OSK_GALAXIE_HEBREW (Passed):

OSK now displays correctly with 'Galaxie Hebrew (Mnemonic)' keyboard:

image

@keymanapp-test-bot keymanapp-test-bot bot removed the user-test-required User tests have not been completed label Oct 24, 2025
Copy link
Copy Markdown
Contributor

@sgschantz sgschantz left a comment

Choose a reason for hiding this comment

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

LGTM

@darcywong00 darcywong00 modified the milestones: A19S14, A19S15 Oct 24, 2025
@mcdurdin mcdurdin merged commit b653f32 into master Oct 27, 2025
8 checks passed
@mcdurdin mcdurdin deleted the fix/mac/15007-ignore-ansi-kvk branch October 27, 2025 14:41
@github-project-automation github-project-automation bot moved this from Todo to Done in Keyman Oct 27, 2025
@keyman-server
Copy link
Copy Markdown
Collaborator

Changes in this pull request will be available for download in Keyman version 19.0.146-alpha

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

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

bug(mac): galaxie_hebrew_mnemonic osk shows tofu key cap labels

4 participants