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

feat(developer): add &displayMap system store 🗜 #9032

Merged
merged 3 commits into from
Jun 20, 2023

Conversation

mcdurdin
Copy link
Member

@mcdurdin mcdurdin commented Jun 17, 2023

Fixes #9031.
Fixes #1070.
Builds on #9016.

The &displayMap system store adds support for a mapping file that remaps the On Screen Keyboard files -- .kvks and .keyman-touch-layout, at compile time, to new ranges. This is specifically intended to resolve a limitation of unattached marks and diacritics which display inconsistently across platforms; this is described in more detail in issue #9031.

In draft because this still needs work for KMW compiler -- but need to do some refactoring on KMW compiler, inverting dependency on kmc-kmn so that kmc-kmn calls into kmc-kmw, before this will be achievable.

Related PRs:

TODO:

@keymanapp-test-bot skip (testing will be in keyboards PR, once 17.0 tooling is in place)

The &displayMap system store adds support for a mapping file that remaps
the On Screen Keyboard files -- .kvks and .keyman-touch-layout, at
compile time, to new ranges. This is specifically intended to resolve a
limitation of unattached marks and diacritics which display
inconsistently across platforms; this is described in more detail in
issue #9031.
@mcdurdin mcdurdin added this to the A17S15 milestone Jun 17, 2023
@keymanapp-test-bot keymanapp-test-bot bot added the user-test-missing User tests have not yet been defined for the PR label Jun 17, 2023
@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented Jun 17, 2023

User Test Results

Test specification and instructions

User tests are not required

Copy link
Contributor

@jahorton jahorton left a comment

Choose a reason for hiding this comment

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

Saw a few things I just wanted to be sure are tracked in some fashion. Aside from that, LGTM.

Base automatically changed from feat/developer/analyze-char-use-continued to feature-kmc-kmw June 20, 2023 06:01
@mcdurdin mcdurdin merged commit 198ba46 into feature-kmc-kmw Jun 20, 2023
@mcdurdin mcdurdin deleted the feat/developer/kmc-display-map-store branch June 20, 2023 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants