-
-
Notifications
You must be signed in to change notification settings - Fork 39.8k
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 flipped and discrete encoder support to PlaidPad default mappings #9396
Conversation
I will leave this as a draft until I get feedback from @BenRoe, no rush though. |
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.
Added some inline comments.
addressed typo and leaving the encoder rotation for now based on our chat. ccw -> left/down |
@matthewweldon please add the changes in the config.h from here to this pr. |
done, @BenRoe, I think this is ready to merge, going to make a new branch for the next keymap later |
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 to me.
Are there some questions about this pr, or is something missing? Please merge it. |
Thanks! |
* Add flipped and discrete encoder support default mappings * Fixed typo in readme * Removed flipped layout, fix config.h encoder pin mapping, update readme Co-authored-by: matt <[email protected]>
The default mapping for the keycapsss plaid pad 4x4 does not include support for different keypresses on the rotary encoders, and doesn't work for encoder hardware that have flipped orientations.
These changes address those issues by adding a new flipped layout and adding left/right arrows to the left encoder.
@BenRoe should approve this PR before merge
Description
This PR provides two changes.
The
default
plaid pad mapping will have left/right arrows mapped to the left encoder, and keeps up/down arrows mapped to the right encoder. Keymapreadme.md
updated to reflect this change.A new keymap is added for the plaid pad called
default_flipped
which adds the flipped encoder support to the config.h file and also keeps the arrow changes from the first change. Keymapreadme.md
added for explaining its usage.Types of Changes
Checklist