-
-
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 royal directory and add 2 keyboard releases #8877
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.
Mostly some indentation and alignment stuff.
I made a comment re: a double-tapdance on both KC_QUOTE
and KC_DQT
-- I'm not certain KC_DQT
will work there as it's a shifted character.
I also changed the indexing on the LAYOUT macros. Not a stopper for me, but I think it's clearer with 0-based indexing on both rows and columns, and hex digits for the higher column numbers.
Co-Authored-By: ridingqwerty <[email protected]>
Co-Authored-By: ridingqwerty <[email protected]>
Co-Authored-By: ridingqwerty <[email protected]>
Co-Authored-By: ridingqwerty <[email protected]>
new Co-Authored-By: ridingqwerty <[email protected]>
@ridingqwerty Okay why is this failing now lol |
I guess its because I deleted my How do I format that?
Like this? |
Cool, I Fixed it :D |
Nice. I'm a little stumped why that mucked with get_tapping_term, which is implemented in action_tapping.c, and action_tapping.h has the core define for TAPPING_TERM. I'll probably back off suggesting people remove that setting when it's 200. |
Co-Authored-By: Drashna Jaelre <[email protected]>
@zvecr Whats does the |
sorry that was a bad click... |
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.
Also, should put a rules.mk
alongside keyboards/the_royal/schwann/keymaps/default/keymap.c
with TAP_DANCE_ENABLE = yes
, so it compiles.
Cool got it all up to date |
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.
Thanks! |
Awesome! Thanks again for everything you guy do! @ridingqwerty @zvecr @noroadsleft @drashna @fauxpark |
* add royal project folder and 2 released keyboards * fix iso keymap clarification * Apply suggestions from code review * Update keyboards/the_royal/schwann/schwann.h * Update keyboards/the_royal/liminal/liminal.h * Update keyboards/the_royal/schwann/keymaps/default/keymap.c * Apply suggestions from code review new * Update keymap.c * Update keymap.c * Apply suggestions from code review * Update rules.mk * add rules.mk to schwann default * Apply suggestions from code review * add rules.mk to shwann default
* add royal project folder and 2 released keyboards * fix iso keymap clarification * Apply suggestions from code review * Update keyboards/the_royal/schwann/schwann.h * Update keyboards/the_royal/liminal/liminal.h * Update keyboards/the_royal/schwann/keymaps/default/keymap.c * Apply suggestions from code review new * Update keymap.c * Update keymap.c * Apply suggestions from code review * Update rules.mk * add rules.mk to schwann default * Apply suggestions from code review * add rules.mk to shwann default
* add royal project folder and 2 released keyboards * fix iso keymap clarification * Apply suggestions from code review * Update keyboards/the_royal/schwann/schwann.h * Update keyboards/the_royal/liminal/liminal.h * Update keyboards/the_royal/schwann/keymaps/default/keymap.c * Apply suggestions from code review new * Update keymap.c * Update keymap.c * Apply suggestions from code review * Update rules.mk * add rules.mk to schwann default * Apply suggestions from code review * add rules.mk to shwann default
* add royal project folder and 2 released keyboards * fix iso keymap clarification * Apply suggestions from code review * Update keyboards/the_royal/schwann/schwann.h * Update keyboards/the_royal/liminal/liminal.h * Update keyboards/the_royal/schwann/keymaps/default/keymap.c * Apply suggestions from code review new * Update keymap.c * Update keymap.c * Apply suggestions from code review * Update rules.mk * add rules.mk to schwann default * Apply suggestions from code review * add rules.mk to shwann default
* add royal project folder and 2 released keyboards * fix iso keymap clarification * Apply suggestions from code review * Update keyboards/the_royal/schwann/schwann.h * Update keyboards/the_royal/liminal/liminal.h * Update keyboards/the_royal/schwann/keymaps/default/keymap.c * Apply suggestions from code review new * Update keymap.c * Update keymap.c * Apply suggestions from code review * Update rules.mk * add rules.mk to schwann default * Apply suggestions from code review * add rules.mk to shwann default
* add royal project folder and 2 released keyboards * fix iso keymap clarification * Apply suggestions from code review * Update keyboards/the_royal/schwann/schwann.h * Update keyboards/the_royal/liminal/liminal.h * Update keyboards/the_royal/schwann/keymaps/default/keymap.c * Apply suggestions from code review new * Update keymap.c * Update keymap.c * Apply suggestions from code review * Update rules.mk * add rules.mk to schwann default * Apply suggestions from code review * add rules.mk to shwann default
Description
Types of Changes
Issues Fixed or Closed by This PR
Checklist