Skip to content
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

Characters ' " & < > are not working on some Mac keyboards #10

Closed
27 tasks done
Trondtr opened this issue Jun 7, 2023 · 5 comments
Closed
27 tasks done

Characters ' " & < > are not working on some Mac keyboards #10

Trondtr opened this issue Jun 7, 2023 · 5 comments
Labels
bug Something isn't working

Comments

@Trondtr
Copy link

Trondtr commented Jun 7, 2023

See a bug report under keyboard-sme for a discussion. Synopsis:

For a handful of keyboards, the five characters ' " & < >are not produced when pressing the designated key. The fields in the on-screen keyboard on my Mac are blank, and there is no output. For another handful, everything works. Seen from my perspective, the respective .yaml files are identical (modulo some change in position, but the characters behave as indicated irrespective of position, as explained in the original bug message), and I am thus not in a position to say what is going on. All five characters are reserved characters in xml. The following keyboards are broken (checkbox checked if fixed/tested as ok):

Production

(Could be production only for mobile, not desktop)

  • crk: Plains Cree
  • mdf: Moksha
  • myv: Ersä
  • sma: South Saami
  • sme: North Saami
  • smj: Lule Saami
  • smn: Inari Saami
  • sms: Skolt Saami
  • vro: Võro (only &)

Alpha

  • hdn: Northern Haida
  • kpv: Komi Zyrian
  • mns: Mansi
  • mrj: Hill Mari
  • nno: Runic Nynorsk
  • srs: Tsuut'ina (Sarsi)

Experimental

  • bla: Siksika
  • dag: Dagbani (NB! E row = nob, not eng)
  • fkv: Kven
  • ike: Eastern Canadian Inuktitut
  • kio: Kiowa
  • krl: Karelian
  • lut: Lushootseed
  • olo: Livvi Karelian
  • rom: Romany
  • tau: Upper Tanana
  • tqn: Sahaptin Tenino
  • urj: FUPA
@snomos snomos changed the title Characters ' " & are rendered in some keyboards, not in others. Characters ' " & are rendered in some keyboards, not others. Jun 8, 2023
@snomos snomos changed the title Characters ' " & are rendered in some keyboards, not others. Characters ' " & < > are rendered in some keyboards, not others on macOS Jun 8, 2023
@snomos snomos changed the title Characters ' " & < > are rendered in some keyboards, not others on macOS Characters ' " & < > are rendered in some keyboards, not others on mac Jun 8, 2023
@snomos
Copy link
Member

snomos commented Jun 8, 2023

I installed the latest nightly version of sme, smn and olo using Divvun Manager, and could reproduce the bug. In addition to the characters mentioned above, also < > are not working, ie producing empty output despite being defined in the yaml file.

The total set of problem letters is thus ' " & < >, all and only the reserved characters in XML.

@alvskar
Copy link
Contributor

alvskar commented Jun 9, 2023

Having a bit of a deja vu, I recall this being an issue sometime in May (2022), getting corrected, and now it seems to be back again.

Looking at the generation it's currently producing results like this:

    <keyMap index="1">      <key code="10" output="§" />
      <key code="18" output="!" />
      <key code="19" output="&quot;" />
      <key code="20" output="#" />
      <key code="21" output="$" />
      <key code="23" output="%" />
      <key code="22" output="&amp;" />
      <key code="26" output="/" />

Key code 19 and 22 are probably confusing to MacOS here. I believe the appropriate codes would be in hex format, i.e., &#x0022; and &#x0026;.

I think the issue is within the xmlem library and how it decides to print attribute values. There were some additional changes done to escaping for xml there and it may not be interpreting some cases as Hex. And ' just seems to have been forgotten. Angle brackets probably printed as is, confusing xml, and getting eaten.

Should be able to fix this next week.

@snomos
Copy link
Member

snomos commented Jun 13, 2023

This is fixed for the SME keyboards after a rebuild. New version is 1.0.5, but it is only available in the nightly channel ATM. I'll keep this issue open until all affected keyboards have been fixed and released.

@Trondtr
Copy link
Author

Trondtr commented Jun 14, 2023

Now, sme works for me as well -- great!
But there are more not working than my 50/50 list. I went through all keyboards in the Divvun Manager and checked them all (I hope). Here are the ones that did not work. Some of them were not implemented, but contained the sme setup only, these I have marked as such.

SME layout:

  • esu: Central Alaskan Yupik
  • gur: Farefare (Frafra)
  • liv: Livonian (sme with small letters only)
  • mhr: Meadow Mari
  • udm: udmurt (sme with small letters only)
  • apu: Apuriña
  • kca: khanty (sme with small letters only)

These layouts will not be fixed now. At some point they will be worked on, to replace the sme layout with a proper one for the target language, and at that point, this bug will automatically be fixed.

Layouts with this bug that are in actual use or have a real layout have been listed in comment 1.

@snomos snomos changed the title Characters ' " & < > are rendered in some keyboards, not others on mac Characters ' " & < > are not working on some Mac keyboards Jun 14, 2023
@snomos snomos changed the title Characters ' " & < > are not working on some Mac keyboards Characters ' " & < > are not working on some Mac keyboards Jun 14, 2023
@snomos
Copy link
Member

snomos commented Aug 14, 2023

All layouts listed in comment 1 have now been fixed. I am closing this bug.

If new keyboards are found with this bug, either just fix them by triggering a rebuild (bumping the bug fix number is enough), or file new bug reports.

@snomos snomos closed this as completed Aug 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants