keymanapp / keyman

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

bug(developer): kmc-keyboard-info should not list web support for multi-keyboard packages #11645

Open mcdurdin opened 4 months ago

mcdurdin commented 4 months ago

I wonder also about the bundle packages (e.g. galaxie_greek_hebrew_mnemonic) if there may be a separate issue to tackle here? It seems like we are adding support for web platforms but the bundled name will not be available in web.

This probably needs to be tackled on the compiler side. A little more nuance may be needed to exclude web platforms from multi-keyboard packages.

Originally posted by @mcdurdin in https://github.com/keymanapp/keyboards/issues/2793#issuecomment-2143621726

See also keymanapp/keyboards#2423

mcdurdin commented 3 months ago

https://downloads.keyman.com/keyboards/galaxie_greek_hebrew_positional/2.2.1/ is clearly wrong: .js filename does not match folder name:

image

This is coming up because there are .js files inside the package, and each .js supports web, but collectively they don't work.