[shughni] New keyboard - #4023
Conversation
|
Thank you for your pull request. You'll see a "build failed" message until the Keyman team has reviewed the pull request and manually initiated the build process. Every change committed to this branch will become part of this pull request. When you have finished submitting files and are ready for the Keyman team to review this pull request, please post a "Ready for review" comment. |
|
Ready for review |
|
Thanks for your Pull Request. I do have some things that need fixing up before we can accept it.
When you add the desktop rules, we may have to modify what is included here. I know this is a lot. But, for a first time contributor I'm very please that all the files are in the right place! |
|
@sharukhhhan do you need help with this pull request? |
|
Hi @LornaSIL! |
e863a1a to
e77b8cd
Compare
|
Hi all! @darcywong00 @LornaSIL |
|
I won't be able to review until next week. I'm sorry.
…On Fri, Jul 17, 2026, 3:24 AM Shokhrukh ***@***.***> wrote:
*sharukhhhan* left a comment (keymanapp/keyboards#4023)
<#4023 (comment)>
Hi all! @darcywong00 <https://github.com/darcywong00> @LornaSIL
<https://github.com/LornaSIL>
Kindly ask you to review it please!
—
Reply to this email directly, view it on GitHub
<#4023?email_source=notifications&email_token=ABV6RI3DDFQQ5KRIR7YC2QL5FHIBRA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKMBQGAYTCOBVGI32M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-5000118527>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABV6RI54NLYYRZ2EJQZ7ZMT5FHIBRAVCNFSNUABEKJSXA33TNF2G64TZHMYTCNBZGU4TONJ3JFZXG5LFHM2DMNBUGE3TAMJRGGQXMAQ>
.
Triage notifications, keep track of coding agent tasks and review pull
requests on the go with GitHub Mobile for iOS
<https://github.com/notifications/mobile/ios/ABV6RI4TCHGCWIEEXQDMYJ35FHIBRA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKMBQGAYTCOBVGI32M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJKTGN5XXIZLSL5UW64Y>
and Android
<https://github.com/notifications/mobile/android/ABV6RI3WBKKP2XYKLHN5LTD5FHIBRA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKMBQGAYTCOBVGI32M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLTGN5XXIZLSL5QW4ZDSN5UWI>.
Download it today!
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
darcywong00
left a comment
There was a problem hiding this comment.
Thank you for the keyboard submission.
I have just a few suggestions for now. Lorna may have more when she returns next week
| { "text": "ê", "id": "U_00EA" } | ||
| ], | ||
| "multitap": [ | ||
| { "text": "ê", "id": "U_00EA" } |
There was a problem hiding this comment.
Several of the touch layout keys seem to duplicate longpress and multitap for the same output.
Since your documentation refers to longpress, I recommend removing the multitap keys (easier to maintain)
There was a problem hiding this comment.
But we wanted to keep both variants, because we found both convenient. We even organized a small focus-group of Pamiri people, and they agreed to keep both variants. Is it okay to keep both?
Co-authored-by: Darcy Wong <darcy_wong@sil.org>
|
I have a few comments which are unrelated to @darcywong00 comment about using multi-tap and longpress. I don't know if there are technical issues you will have with implementing both or not. I do realize some of the languages in the Shughni region have mixed script orthographies (Latin/Greek/Cyrillic). Shughni seems to be a mostly Latin orthography with a few Greek and Cyrillic characters. The more you can standardize, the better. Your uppercase/lowercase will work better in applications and fonts will have the design consistent. Mixing Latin and Greek often means a slightly different style. You are using U+03D1 (lowercase Greek theta) and U+019F (Uppercase Latin theta). Shouldn't you use the uppercase Greek theta U+0398? The problem with mixing pairs is that if someone takes your text, and converts the uppercase greek theta to lowercase they won't get the latin theta (and vice versa). So, it is certainly best if you can use the matching case pair. Using Greek delta (δ / Δ) is a matched pair so I think that's fine. Then, on the I'm not sure what the orthography is meant to be here. If you want a case pair you could use the Cyrillic character instead of 0293. or if you really want Cyrillic, use ӡӠ, But again, mixing 0293 and 04E1 seems like a mistake and your users won't be too happy. Lastly, once you make any changes, could you reimport the On-screen keyboard? Make sure to select "Auto-fill underlying layout" first and then select "Fill from layout". That will make sure all the digits and punctuation get included in the On-Screen keyboard so people will know what to type. |
This pull request is for adding the Shughni language (one of the Pamiri ones) to the Keyman database of languages