Skip to content

Soninke N'ti - removed unused punctuation#3782

Merged
LornaSIL merged 5 commits into
keymanapp:masterfrom
oyousuf:nti
Nov 18, 2025
Merged

Soninke N'ti - removed unused punctuation#3782
LornaSIL merged 5 commits into
keymanapp:masterfrom
oyousuf:nti

Conversation

@oyousuf
Copy link
Copy Markdown

@oyousuf oyousuf commented Nov 11, 2025

Removed 2 punctuation marks that the community told me they aren't using. Added 1 other punctuation mark (inverted exclamation mark) that they are using. Fixed one glyph error in the font. Updated typing manual to reflect these changes.

…ther punctuation mark being used. Documentation updated accordingly.
@keyman-server
Copy link
Copy Markdown
Collaborator

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.

@LornaSIL
Copy link
Copy Markdown
Contributor

  • Update HISTORY.md so it is like the following. The goal is to have the version and date with the changes below it.
1.0.2 (2025-11-11)
----------------
* Removed 2 punctuation marks not used by people. Added exclamation mark used by people.

1.0.1 (2025-05-29)
----------------
* Added keystrokes to output ordinal numbers.

1.0 (2024-07-23)
----------------
* Created by Oreen Yousuf

  • Can you undo the change to the php file? The online help files don't look great with <h1> elements because the title of the page is an h1 and it's unnecessary to include the title "Keyboard Layout". If you feel it's necessary, you also need to fix the </h2? to </h2>
  • You'll need to remove the temporary file that got added: ~$iTypingManual.docx (that is a transient file in Windows when you have the Word document opened)
  • If you want to commit your Word document which is used for the pdf, you may, but we would prefer you put it in an extras folder so we know it's not part of the package.

@oyousuf
Copy link
Copy Markdown
Author

oyousuf commented Nov 15, 2025

Thanks for catching those errors. I think I fixed them all now.

Updated changes so it's clear what version is affected.
Fix h2 tag to be properly formed.
Copy link
Copy Markdown
Contributor

@LornaSIL LornaSIL left a comment

Choose a reason for hiding this comment

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

LGTM

@LornaSIL LornaSIL merged commit ee25464 into keymanapp:master Nov 18, 2025
2 checks passed
@oyousuf oyousuf deleted the nti branch November 19, 2025 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants