-
-
Notifications
You must be signed in to change notification settings - Fork 40.6k
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 incorrect layout for GHS.rar #9597
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some small formatting tweaks for readability. Could you also do the same for the VIA keymap? I can't make a suggestion on the first line of keycodes as it's out of range.
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Thanks! I'll get working on the changes right away. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks!
* 'master' of https://github.com/qmk/qmk_firmware: (154 commits) Fix typo in freebsd_install.sh (qmk#9655) [Keymap] Add Alice rev2 to my userspace (qmk#9600) [Keyboard] RART67 : add support via (qmk#9589) [Keyboard] XD87 Caps Lock LED refactor (qmk#9585) [Keyboard] Updated VIA Support for DP60 (qmk#9508) fixed link to Eclipse instructions (qmk#9647) format code according to conventions [skip ci] add DIP_SWITCH_MATRIX_GRID support (qmk#8772) add SPLIT_HAND_MATRIX_GRID support (qmk#8685) [Keyboard] Gowla Macro Board (qmk#9643) Fix “G” in Neo2 keymap (qmk#9645) Rotr - Updated README and default keymap (qmk#9598) Add missing dependency for Void Linux to util/linux-install.sh (qmk#9637) Backlight docs wordsmithing (qmk#9631) Fix incorrect layout for GHS.rar (qmk#9597) Add movement hook to ps2_mouse (qmk#8805) Make sendstring respect `TAP_CODE_DELAY` (qmk#9623) [Keymap] Adding a Custom Keymap for Matrix Noah (qmk#9581) Fix sharing of mouse button state from mousekeys to ps2_mouse (qmk#9124) Add xjtian keymap for Planck (qmk#9502) ...
Co-authored-by: Ryan <fauxpark@gmail.com>
Description
Fixes the layout for GHS.rar as pointed out in this comment.
Types of Changes
Issues Fixed or Closed by This PR
Previously, it would compile the firmware even if the layout was missing RShift. I've added the RShift to the layouts and removed the commas at the ends of the
LAYOUT_*
macros.Checklist