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

Add macOS-friendly keymap for GMMK Pro (ANSI) #14333

Merged
merged 4 commits into from
Mar 9, 2022

Conversation

zvuc
Copy link

@zvuc zvuc commented Sep 6, 2021

Description

GMMK Pro is a great entry keyboard for many people getting into the hobby (including me), and I'm pretty sure there are many macOS users out there although we're a minority in the Custom Mech Keys world. While the keyboard is usable in macOS out-of-the-box, Glorious does not (yet) provide their Glorious Core software for macOS; so with stock firmware, there aren't any ways to customize layouts on the hardware level unless you turn to software-based override solutions such as Karabiner Elements. This is my attempt to try to create the most macOS-like experience on the GMMK Pro, taking close notes from Apple's Magic Keyboard, also referencing code from other community members.

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Checklist

  • My code follows the code style of this project: C, Python
  • I have read the PR Checklist document and have made the appropriate changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@zvuc zvuc changed the title Add macOS keymap for GMMK Pro (ANSI) Add macOS-friendly keymap for GMMK Pro (ANSI) Sep 6, 2021
@github-actions github-actions bot added the keymap label Sep 6, 2021
@zvuc zvuc force-pushed the gmmk-pro-ansi-zvuc branch 2 times, most recently from 753379c to e9a8321 Compare September 6, 2021 16:00
Copy link
Member

@zvecr zvecr left a comment

Choose a reason for hiding this comment

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

keymap folder name needs to be changed to lowercase

@zvuc zvuc requested a review from zvecr September 7, 2021 01:07
@zvuc zvuc marked this pull request as draft September 7, 2021 02:33
@zvuc
Copy link
Author

zvuc commented Sep 7, 2021

I found a few places I want to change / improve so I'll request a review again after adding a few more commits. Sorry for the confusion!

@zvuc zvuc marked this pull request as ready for review September 13, 2021 13:55
@drashna drashna requested a review from a team September 15, 2021 06:29
@zvuc
Copy link
Author

zvuc commented Oct 8, 2021

Hi, just checking in here again after a while. Is there anything else I need to do regarding this PR to get this merged to master? 😄 @zvecr

@stale
Copy link

stale bot commented Jan 9, 2022

Thank you for your contribution!
This pull request has been automatically marked as stale because it has not had activity in the last 45 days. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, or re-open when it's ready.
For maintainers: Please label with awaiting review, breaking_change, in progress, or on hold to prevent the issue from being re-flagged.

Copy link
Member

@noroadsleft noroadsleft left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@stale stale bot removed the awaiting changes label Mar 9, 2022
@noroadsleft noroadsleft merged commit caeb213 into qmk:master Mar 9, 2022
@noroadsleft
Copy link
Member

Thanks!

@zvuc zvuc deleted the gmmk-pro-ansi-zvuc branch May 18, 2022 15:34
0xcharly pushed a commit to Bastardkb/bastardkb-qmk that referenced this pull request Jul 4, 2022
* Add macOS keymap for GMMK Pro (ANSI)

* Change macOS keymap directory name to lowercase

* Add toggleable layer with alternative keymap for function row

* Update readme
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants