Skip to content

Add keyboard support for Tsishingini (tsw) - #860

Open
ToluAyodele wants to merge 3 commits into
wikimedia:masterfrom
ToluAyodele:T428848
Open

Add keyboard support for Tsishingini (tsw)#860
ToluAyodele wants to merge 3 commits into
wikimedia:masterfrom
ToluAyodele:T428848

Conversation

@ToluAyodele

Copy link
Copy Markdown
Contributor

@ToluAyodele
ToluAyodele requested a review from srish August 18, 2026 20:07

@srish srish left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ToluAyodele Thanks Tolu for the PR! I've left some comments inline.

Besides these comments, there is one Git related issue. It looks like your git author meta data might be broken: “ToluAyo” <“tayodele-ctr@wikimedia.org”> - name and email are wrapped in curly quotes, so the email is invalid and won't attribute. Every of your other commit uses ToluAyodele <129562437+ToluAyodele@users.noreply.github.com>. Might need a git commit --amend --author=.....

Comment thread rules/tsw/tsw-tilde.js
license: 'GPLv3',
version: '1.0',
patterns: [
// Special letters

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like the source document lists several more letters that aren't typeable on a standard keyboard https://www.omniglot.com/charts/kambari.pdf. Is it possible to add those?

Comment thread src/jquery.ime.inputmethods.js
name: 'ChiTumbuka tilde',
source: 'rules/tum/tum-tilde.js'
},
'tsw-tilde': {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same sorting issue here too. Goes between tn-tilde and tum-tilde.

Comment thread rules/tsw/tsw-tilde.js Outdated
Comment thread test/jquery.ime.test.fixtures.js Outdated
Comment thread rules/tsw/tsw-tilde.js Outdated
Added patterns for special letters 'c' and 'C' with their respective replacements.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants