Skip to content

fix(web): define IME interfaces in KeyboardInterfaceBase#16170

Open
mcdurdin wants to merge 2 commits into
masterfrom
fix/web/16144-define-interfaces-for-imes-in-base
Open

fix(web): define IME interfaces in KeyboardInterfaceBase#16170
mcdurdin wants to merge 2 commits into
masterfrom
fix/web/16144-define-interfaces-for-imes-in-base

Conversation

@mcdurdin

@mcdurdin mcdurdin commented Jul 1, 2026

Copy link
Copy Markdown
Member

Ensure that the IME interface methods are all defined for both browser and webview modes. Deprecate 'Element' methods and replace with 'TextStore' to clarify return type. Add deprecation flags where needed and update documentation accordingly.

Fixes: #16144

User Testing

@github-project-automation github-project-automation Bot moved this to Todo in Keyman Jul 1, 2026
@mcdurdin mcdurdin requested a review from ermshiperete July 1, 2026 05:41
@github-actions github-actions Bot added the web/ label Jul 1, 2026
@mcdurdin mcdurdin requested a review from jahorton July 1, 2026 05:41
@github-actions github-actions Bot added the fix label Jul 1, 2026
@keymanapp-test-bot keymanapp-test-bot Bot added has-user-test user-test-required User tests have not been completed labels Jul 1, 2026
@keymanapp-test-bot keymanapp-test-bot Bot added this to the A19S32 milestone Jul 1, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Merged the compatibility pages into interface folder, because all the functions were part of the interface object.

Ensure that the IME interface methods are all defined for both browser
and webview modes. Deprecate 'Element' methods and replace with
'TextStore' to clarify return type. Add deprecation flags where needed
and update documentation accordingly.

Fixes: #16144
@mcdurdin mcdurdin force-pushed the fix/web/16144-define-interfaces-for-imes-in-base branch from 0663ee4 to 0fc869f Compare July 1, 2026 05:43
@Meng-Heng Meng-Heng assigned Meng-Heng and unassigned Meng-Heng Jul 2, 2026
@Meng-Heng

Copy link
Copy Markdown
Contributor

Hello, @mcdurdin. I don't think this PR has the fixed for "fetching keyboard from server" since I am seeing this when launching the apk. I can revisit to do the testing at another time.

Screenshot 2026-07-02 at 12 30 53 in the afternoon

@mcdurdin

mcdurdin commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

Hello, @mcdurdin. I don't think this PR has the fixed for "fetching keyboard from server" since I am seeing this when launching the apk. I can revisit to do the testing at another time.

Right, I've just merged in the patch from master so the new build on this branch should work I hope!

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

Labels

fix has-user-test user-test-required User tests have not been completed web/

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

bug(web): error displayed when typing with korean_rr keyboard in Keyman for iOS or Keyman for Android

2 participants