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.

Sign up for GitHub

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

Jump to bottom

Updated layout, added xkb 2.39 notice #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Updated layout, added xkb 2.39 notice #19

wants to merge 1 commit into from

Conversation

cs127
Copy link
Contributor

@cs127 cs127 commented Mar 20, 2023 •
edited
Loading

  • Changed layout ID from ir(pes_winkeys) to ir(winkeys) to match the latest commit in xkb config.
  • Changed the number keys on row E to type ASCII Latin digits instead of Persian digits.
  • Added Persian digits to the third level of the number keys on row E.
  • Added Unicode characters ZWJ, ZWNJ, LRM, and RLM to the fourth level of the keys AE01 - AE04.
  • Added the letter Pe (U+067E) to the LSGT key, located to the left of AB01 on some keyboards.
  • Removed the nbsp inclusion.
  • Added a notice about xkb config v2.39.

@cs127 cs127 mentioned this pull request Mar 20, 2023
Open
@cs127 cs127 force-pushed the master branch 2 times, most recently from c45f4fa to daeb408 Compare March 20, 2023 13:09
@cs127 cs127 mentioned this pull request Apr 30, 2023
Open
@cs127 cs127 mentioned this pull request May 24, 2023
Closed
MasterKia
MasterKia reviewed Sep 7, 2023
View reviewed changes
README.md
@@ -43,7 +43,7 @@ cd windows-persian-keyboard-for-linux

```bash
sudo cp /usr/share/X11/xkb/symbols/ir /usr/share/X11/xkb/symbols/ir.backup
sudo cat ./ir | sudo tee -a /usr/share/X11/xkb/symbols/ir &>/dev/null
cat ./ir_patch | sudo tee -a /usr/share/X11/xkb/symbols/ir >/dev/null
Copy link

Choose a reason for hiding this comment

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

Yeah. This should be merged because currently the manual installation is broken.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MasterKia MasterKia MasterKia left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cs127 @MasterKia