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.

Sign up for GitHub

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

Jump to bottom

feat(developer): visual LDML editor prototype 🗼 #12789

Open

srl295 opened this issue Dec 5, 2024 · 0 comments

Assignees

Labels

developer/ epic-ldml-editor feat

Milestone

Member

srl295 commented Dec 5, 2024

Spec:

spec(developer): LDML Keyboard Designer 🗼 #5016

Develop a visual editor for LDML.
Add a webview to the VSCode plugin that does a basic load of the XML file and displays some initial UI elements reflecting the contents of the keyboard.

Blocked by:

feat(developer): VSCode Plugin for kpj/xml/kmn builds 🗼 #12756

srl295 added the epic-ldml-editor label

srl295 added this to the A18S16 milestone

srl295 self-assigned this

srl295 added this to Keyman

keymanapp-test-bot bot added developer/ feat labels

srl295 added a commit that referenced this issue


          feat(developer): seed ldml visual editor 🗼

eb7c407

- minor cleanup to re-add the custom view
- some comments

Fixes: #12789

srl295 mentioned this issue

feat(developer): prototype of ldml visual editor 🗼 #12790

Draft

srl295 added a commit that referenced this issue


          chore(developer): add react/parcel 🗼

6c62ecc

- not quite glued in properly yet

Fixes: #12789

srl295 added a commit that referenced this issue


          chore(developer): add react/parcel 🗼

8f354f7

- Glued in now. Or at least duct taped.
- parcel builds into build/ which is picked up by the app

Fixes: #12789

srl295 mentioned this issue

feat(developer): scaffolding for VSCode plugin 🗼 #12798

Open

srl295 added a commit that referenced this issue


          feat(developer): seed ldml visual editor 🗼

a4899a3

- minor cleanup to re-add the custom view
- some comments

Fixes: #12789

srl295 added a commit that referenced this issue


          chore(developer): add react/parcel 🗼

ccd2d54

- add React 19
- parcel builds into build/ which is picked up by the app

Fixes: #12789

srl295 added a commit that referenced this issue


          feat(developer): seed ldml visual editor 🗼

9b363d4

- minor cleanup to re-add the custom view
- some comments

Fixes: #12789

srl295 added a commit that referenced this issue


          chore(developer): add react/parcel 🗼

a3dcba3

- add React 19
- parcel builds into build/ which is picked up by the app

Fixes: #12789

darcywong00 modified the milestones: A18S16, A18S17

srl295 added a commit that referenced this issue


          chore(developer): rename test file🗼

- per code review

Fixes: #12789

srl295 added a commit that referenced this issue


          chore(developer): package🗼

97ff64c

- per code review

Fixes: #12789

srl295 added a commit that referenced this issue


          chore(developer): update build script🗼

3b36e05

- per code review

Fixes: #12789

srl295 added a commit that referenced this issue


          feat(developer): seed ldml visual editor 🗼

e7bf987

- minor cleanup to re-add the custom view
- some comments

Fixes: #12789

srl295 added a commit that referenced this issue


          chore(developer): add react/parcel 🗼

ac57afe

- add React 19
- parcel builds into build/ which is picked up by the app

Fixes: #12789

srl295 added a commit that referenced this issue


          feat(developer): seed ldml visual editor 🗼

cea397e

- minor cleanup to re-add the custom view
- some comments

Fixes: #12789

srl295 added a commit that referenced this issue


          chore(developer): add react/parcel 🗼

15c0441

- add React 19
- parcel builds into build/ which is picked up by the app

Fixes: #12789

srl295 added a commit that referenced this issue


          chore(developer): update README and package.json 🗼

3ec6f85

- build fix

Fixes: #12789

srl295 added a commit that referenced this issue


          chore(developer): ldml-editor: unbreak: add version number 🗼

1bb3011

- without this, the extension inexplicably stops working
- This probably got dropped in the merge
- Was noted in #12835 but wasn't clear this blocked extension operation

Fixes: #12789

srl295 added a commit that referenced this issue


          chore(developer): ldml-editor: unbreak: add version number 🗼

d758a3b

- without this, the extension inexplicably stops working
- This probably got dropped in the merge
- Was noted in #12835 but wasn't clear this blocked extension operation

Fixes: #12789

darcywong00 modified the milestones: A18S17, A18S18

srl295 added a commit that referenced this issue


          chore(developer): ldml-editor: use NodeCompilerCallbacks 🗼

079270f

- add an extension point to NodeCompilerCallbacks to support changing message target
- add an LDMLCompilerManager to simplify the CommonJS/ESM boundary

Fixes: #12789

srl295 added a commit that referenced this issue


          chore(developer): update for update from master

b2167fe

- use correct imports paths
- user gets a fatal error if the keyboard file doesn't compile. But at least it compiles!

Fixes: #12789

srl295 added a commit that referenced this issue


          feat(developer): update React side to show if the keyboard loaded or not

ed94bba

Fixes: #12789

srl295 added a commit that referenced this issue


          feat(developer): use index.html from parcel

a8fd0dc

- fixup index.html from parcel and use it. this avoids hard coded hex ids
- also, fix vscode-test.mjs to support correct .tests. name

Fixes: #12789

srl295 added a commit that referenced this issue


          feat(developer): initial views 🗼

e060507

- keybag
- layer list

Fixes: #12789

srl295 added a commit that referenced this issue


          feat(developer): initial views 🗼

900ec9a

- keybag
- layer list with key ids

Fixes: #12789

srl295 added a commit that referenced this issue


          feat(developer): use react context 🗼

7ba82fb

Fixes: #12789

srl295 added a commit that referenced this issue


          feat(developer): use index.html from parcel

ea7dc33

- add antd
- not sure this is a sustainable way to add css inline content

Fixes: #12789

srl295 added a commit that referenced this issue


          feat(developer): reduce log verbosity

ca78a25

Fixes: #12789

srl295 added a commit that referenced this issue


          feat(developer): abandon hashes for style sheets

685fdfb

Fixes: #12789

srl295 added a commit that referenced this issue


          feat(developer): pass up messages to UI 🗼

57cafb3

Fixes: #12789

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