Skip to content

[hong_kong] keyboard updating#3873

Merged
LornaSIL merged 4 commits intokeymanapp:masterfrom
Cathaylaboratory:master
Feb 12, 2026
Merged

[hong_kong] keyboard updating#3873
LornaSIL merged 4 commits intokeymanapp:masterfrom
Cathaylaboratory:master

Conversation

@Cathaylaboratory
Copy link
Copy Markdown
Contributor

ready for reviewing

@keyman-server
Copy link
Copy Markdown
Collaborator

Thank you for your pull request. You'll see a "build failed" message until the Keyman team has reviewed the pull request and manually initiated the build process.

Every change committed to this branch will become part of this pull request. When you have finished submitting files and are ready for the Keyman team to review this pull request, please post a "Ready for review" comment.

@LornaSIL
Copy link
Copy Markdown
Contributor

On the touch layout there are some undefined keys.

                  {
                    "id": "T_new_686"
                  },
                  {
                    "id": "T_new_687"
                  },
                  {
                    "id": "T_new_688"
                  },
                  {
                    "id": "T_new_689"

I wonder if you need to remove those are what is needed.

The LICENSE.md file should have a copyright date of 2025-2026. It needs to have the beginning date through the year it's revised.
Otherwise, I think it's okay.

@LornaSIL LornaSIL changed the title Hong Kong keyboard updating [hong_kong] keyboard updating Feb 10, 2026
@Cathaylaboratory
Copy link
Copy Markdown
Contributor Author

Cathaylaboratory commented Feb 10, 2026 via email

Copy link
Copy Markdown
Contributor

@LornaSIL LornaSIL left a comment

Choose a reason for hiding this comment

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

LGTM

@LornaSIL LornaSIL merged commit 12e8c32 into keymanapp:master Feb 12, 2026
2 checks passed
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.

3 participants