keymanapp / keyman

Keyman cross platform input methods system running on Android, iOS, Linux, macOS, Windows and mobile and desktop web
https://keyman.com/
Other
386 stars 107 forks source link

chore(web): move KeyboardProcessor.Codes into common/web/types #8146

Closed mcdurdin closed 1 month ago

mcdurdin commented 1 year ago

I suggest that we move codes.ts into common/web/types as a future refactor.

_Originally posted by @mcdurdin in https://github.com/keymanapp/keyman/pull/7809#discussion_r1094022876_

Also, SystemStoreIDs from kbdInterface.ts

jahorton commented 1 year ago

An extension of this, coming out of the big namespace -> module feature branch that just landed (#8560):

@keymanapp/keyboard-processor is practically primed for a split into two components:

There may be a slight bit of work needed, but I remember noticing how split-friendly the package's organization & contents were while converting KMW to its now-ES-module-based form.

jahorton commented 1 year ago

https://github.com/keymanapp/keyman/pull/8056 made related changes in preparation for this - it was made at the time I identified the potential split and started more actively preparing for it.

ermshiperete commented 2 months ago

After discussion moving to A18S7 when @mcdurdin is scheduled to be back.