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.

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

Fix #1531: Revert control key remapping from grave/tilde to zenkaku #1541

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

Conversation

vladimirlagunov
Copy link

@vladimirlagunov vladimirlagunov commented Jan 26, 2022

The bug was introduced in the pull request #1214, and even in the description of the request it had been acknowledged that there is no such key on mac keyboards.

As a consequence, it led to inability to transfer the grave accent and and the tilde, at least from macOS to Windows on default "ABC" keyboard layout.

Unfortunately, I don't have a JIS keyboard, nor understand how to use it. The only thing I can suggest now is to revert the controversial remapping.

Contributor Checklist:

  • I have created a file in the doc/newsfragments directory IF it is a
    user-visible change (and make sure to read the README.md in that directory)

…zenkaku

The bug was introduced in the pull request debauchee#1214, and even in the description of the request it had been acknowledged that there is no such key on mac keyboards.

As a consequence, it led to inability to transfer the grave accent and and the tilde, at least from macOS to Windows on default "ABC" keyboard layout.

Unfortunately, I don't have a JIS keyboard, nor understand how to use it. The only thing I can suggest now is to revert the controversial remapping.
@dgentry
Copy link

dgentry commented Jul 7, 2022

Because this PR stalled for reasons I couldn't see, I created a new PR, #1711, that fixes the Azure breakage for Mojave builds. (It turns out the checks on this PR failed because Microsoft removed the MacOS 10.14 image from Azure).

I also created a working Barrier.app available here, although so far I've only built it for m1 macs running Monterey (in the hope that PR #1711 or this PR will be accepted soon).

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.

None yet

2 participants