-
-
Notifications
You must be signed in to change notification settings - Fork 40.5k
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 rainkeeb keyboard #10875
Add rainkeeb keyboard #10875
Conversation
Add support for rainkeeb
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.
I reworked the LAYOUT macro to a bit more readable and user-friendly in the keymap.
Also did some suggestions for current conventions and formatting.
keyboards/rainkeeb/info.json
Outdated
{"label":"k00", "x":0, "y":0}, | ||
{"label":"k01", "x":1, "y":0}, | ||
{"label":"k02", "x":2, "y":0}, | ||
{"label":"k03", "x":3, "y":0}, | ||
{"label":"k04", "x":4, "y":0}, | ||
{"label":"k05", "x":7, "y":0}, | ||
{"label":"k06", "x":8, "y":0}, | ||
{"label":"k07", "x":9, "y":0}, | ||
{"label":"k08", "x":10, "y":0}, | ||
{"label":"k09", "x":11, "y":0}, | ||
{"label":"k10", "x":0, "y":1}, | ||
{"label":"k11", "x":1, "y":1}, | ||
{"label":"k12", "x":2, "y":1}, | ||
{"label":"k13", "x":3, "y":1}, | ||
{"label":"k14", "x":4, "y":1}, | ||
{"label":"k15", "x":7, "y":1}, | ||
{"label":"k16", "x":8, "y":1}, | ||
{"label":"k17", "x":9, "y":1}, | ||
{"label":"k18", "x":10, "y":1}, | ||
{"label":"k19", "x":11, "y":1}, | ||
{"label":"k20", "x":0, "y":2}, | ||
{"label":"k21", "x":1, "y":2}, | ||
{"label":"k22", "x":2, "y":2}, | ||
{"label":"k23", "x":3, "y":2}, | ||
{"label":"k24", "x":4, "y":2}, | ||
{"label":"k25", "x":7, "y":2}, | ||
{"label":"k26", "x":8, "y":2}, | ||
{"label":"k27", "x":9, "y":2}, | ||
{"label":"k28", "x":10, "y":2}, | ||
{"label":"k29", "x":11, "y":2}, | ||
{"label":"k30", "x":0, "y":3}, | ||
{"label":"k31", "x":1, "y":3}, | ||
{"label":"k32", "x":2, "y":3}, | ||
{"label":"k33", "x":3, "y":3, "w":2}, | ||
{"label":"k34", "x":5, "y":3, "w":2}, | ||
{"label":"k35", "x":7, "y":3, "w":2}, | ||
{"label":"k36", "x":9, "y":3}, | ||
{"label":"k37", "x":10, "y":3}, | ||
{"label":"k38", "x":11, "y":3} |
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.
This will match the updated LAYOUT in rainkeeb.h, but I'm unsure if it's worth reordering the kXY
to match the macro elements..
Co-authored-by: ridingqwerty <[email protected]>
Co-authored-by: ridingqwerty <[email protected]>
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.
You might want to have a TAP_CODE_DELAY
time defined, or some of the encoder functions may not work properly.
they all seem to be working perfectly so far, i don't think adding that is neccesary |
Looks good, thanks! |
* Add rainkeeb Add support for rainkeeb * gpl license added * updated * Apply suggestions from code review Co-authored-by: ridingqwerty <[email protected]> * Apply suggestions from code review Co-authored-by: ridingqwerty <[email protected]> * Update info.json * via keymap added * updated keymap and .h to work properly * fixed oled code oops Co-authored-by: ridingqwerty <[email protected]>
Adds support for rainkeeb keyboard.
Description
Adds support for rainkeeb keybord, my first custom design.
Types of Changes
Issues Fixed or Closed by This PR
Checklist