-
-
Notifications
You must be signed in to change notification settings - Fork 109
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): LDML Keyboard Test Window 🌱 #10459
feat(developer): LDML Keyboard Test Window 🌱 #10459
Conversation
User Test ResultsTest specification and instructions
Test Artifacts
|
Hi @bharanidharanj, that's a Keyman keyboard project, not an LDML keyboard project. To create an LDML keyboard project, you need to select Project / New Project / LDML Keyboard. Can you retest please? Thanks! @keymanapp-test-bot retest all |
@mcdurdin I noticed that there was an error message that appeared in 'Node' widow which is running behind the Keyman Developer application. |
@bharanidharanj (I think the node error is unrelated) Can you follow the steps in the video to run your test keyboard (note, once you have selected Test Keyboard, then you'll need to press a as per the instructions above; my example does not have that final step) new-ldml-keyboard.mp4 |
@mcdurdin Followed the steps as mentioned in the video and I got an error message after clicking the 'Test Keyboard' option. |
The issue is the techpreview files are not included in the Keyman Developer msi. I am looking at how best to solve. |
@mcdurdin should there be some kind of sanity checking at start-up? to make sure the schemas and data all arrived ok? |
Note: schemas are bundled in the .mjs but imports are not. |
…-ldml-keyboard-test
@bharanidharanj the underlying issues have now been addressed, hopefully! Please retest 😁 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seeing all the if ActiveLdmlKeyboardEditor
else if ActiveLdmlKeyboardEditor
feels like some aggregation or maybe even inheritance could have been used for a ActiveKeyboardEditor
. Otherwise
LGTM
Agree. Certainly would be a nice refactor. For now, given the code is deprecated before it even lands, leaving it as is. The ActiveLdmlKeyboardEditor is likely to be completely rewritten in 18.0 as a web component. |
Fixes #10420.
Implements a basic LDML keyboard test window. Press F5 or Keyboard/Test to start test. Shows a character grid for each character in the text.
Does not yet support display of markers (#10458). Does not support interactive debugging, nor any more detailed status information.
User Testing
Hello World!
and that basic text editing functions work as expected. Press Shift+F5 or the stop button in the debug toolbar to stop testing.