Skip to content

Remap F3 key for HIMS displays to avoi conflicts with number 6 in computer braille tables#11710

Merged
michaelDCurran merged 2 commits into
nvaccess:masterfrom
lukaszgo1:HIMS-remap_F3
Oct 1, 2020
Merged

Remap F3 key for HIMS displays to avoi conflicts with number 6 in computer braille tables#11710
michaelDCurran merged 2 commits into
nvaccess:masterfrom
lukaszgo1:HIMS-remap_F3

Conversation

@lukaszgo1

Copy link
Copy Markdown
Contributor

Link to issue number:

None, reported by users on one of the Polish mailing lists and subsequently confirmed by Polish distributor of HIMS displays.

Summary of the issue:

Currently for HIMS braille displays F3 key is assigned to the braille points 1248. For computer braille tables however (at least for the Polish one) dots 1248 should enter number 6. For users it is impossible to write "6" from the braille keyboard in computer braille on HIMS displays.

Description of how this pull request fixes the issue:

F3 key has been remapped to Space+148. If we assume that Space+point 8 is responsible for invoking F keys this mapping is logical since dots 14 represents number 3.

Testing performed:

TBD - I'll send the build to the distributor I don't have display to test.

Known issues with pull request:

None known

Change log entry:

Section: Bug fixes
It is now possible to enter number 6 in computer braille from a braille keyboard on HIMS displays.
Changes:

For HIMS braille displays F3 has been remapped to Space + dots 148

LeonarddeR
LeonarddeR previously approved these changes Oct 1, 2020

@LeonarddeR LeonarddeR left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This change makes sense to me. Actually I blindly copied the gesture mapping from the old driver.

@michaelDCurran michaelDCurran merged commit 780f04d into nvaccess:master Oct 1, 2020
@nvaccessAuto nvaccessAuto added this to the 2020.4 milestone Oct 1, 2020
@lukaszgo1 lukaszgo1 deleted the HIMS-remap_F3 branch October 2, 2020 08:46
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.

4 participants