-
-
Notifications
You must be signed in to change notification settings - Fork 40.1k
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
[keyboard]Graystudio BD60 Support #22210
Conversation
"device_version": "0.0.1", | ||
}, | ||
"features": { | ||
"backlight": false, |
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.
"backlight": false, |
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 still needs removing
Co-authored-by: Joel Challis <[email protected]>
Co-authored-by: Joel Challis <[email protected]>
Co-authored-by: Joel Challis <[email protected]>
Co-authored-by: Joel Challis <[email protected]>
Co-authored-by: Joel Challis <[email protected]>
Co-authored-by: Joel Challis <[email protected]>
Co-authored-by: Joel Challis <[email protected]>
Co-authored-by: Joel Challis <[email protected]>
Co-authored-by: Duncan Sutherland <[email protected]>
Co-authored-by: Duncan Sutherland <[email protected]>
Co-authored-by: Duncan Sutherland <[email protected]>
Co-authored-by: Duncan Sutherland <[email protected]>
Co-authored-by: Duncan Sutherland <[email protected]>
Co-authored-by: Duncan Sutherland <[email protected]>
Co-authored-by: Duncan Sutherland <[email protected]>
Co-authored-by: Duncan Sutherland <[email protected]>
Co-authored-by: Duncan Sutherland <[email protected]>
Co-authored-by: Duncan Sutherland <[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.
4th/Shift row has an extra key (typically ISO Backslash); please remove and correct matrix co-ordinates for remaining keys in said row.
Co-authored-by: Duncan Sutherland <[email protected]>
Co-authored-by: Duncan Sutherland <[email protected]>
…irmware into graystudio-bd60
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.
Please run qmk format-json -i
on this file.
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 is still outstanding, and just to note, there is no need to continually merge master
.
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.
Still outstanding, and this file should also be renamed to keyboard.json
.
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 file should still be removed.
Co-authored-by: Joel Challis <[email protected]>
Co-authored-by: Joel Challis <[email protected]>
"device_version": "0.0.1", | ||
}, | ||
"features": { | ||
"backlight": false, |
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 still needs removing
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 file still needs to be removed
As of August 26, 2024, As you've submitted a PR containing |
"community_layouts": [ | ||
"60_ansi", | ||
"60_ansi_split_bs_rshift", | ||
"60_ansi_tsangan", | ||
"60_tsangan_hhkb", | ||
"60_ansi_wkl", | ||
"60_ansi_wkl_split_bs_rshift", | ||
"60_hhkb" | ||
], |
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.
- make array literal a one liner
- tsangan_hhkb -> ansi_tsangan_split_bs_rshift
"community_layouts": [ | |
"60_ansi", | |
"60_ansi_split_bs_rshift", | |
"60_ansi_tsangan", | |
"60_tsangan_hhkb", | |
"60_ansi_wkl", | |
"60_ansi_wkl_split_bs_rshift", | |
"60_hhkb" | |
], | |
"community_layouts": ["60_ansi", "60_ansi_split_bs_rshift", "60_ansi_tsangan", "60_ansi_tsangan_split_bs_rshift", "60_ansi_wkl", "60_ansi_wkl_split_bs_rshift", "60_hhkb"], |
Thank you for your contribution! |
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.
VIA keymap needs to be removed.
const rgblight_segment_t PROGMEM my_capslock_layer[] = RGBLIGHT_LAYER_SEGMENTS( | ||
{0,1, HSV_WHITE} | ||
); | ||
const rgblight_segment_t* const PROGMEM my_rgb_layers[] = RGBLIGHT_LAYERS_LIST( | ||
my_capslock_layer | ||
); | ||
|
||
void keyboard_post_init_kb(void) { | ||
// Enable the LED layers | ||
rgblight_layers = my_rgb_layers; | ||
|
||
keyboard_post_init_user(); | ||
} | ||
|
||
bool led_update_kb(led_t led_state) { | ||
if (led_update_user(led_state)) { | ||
rgblight_set_layer_state(0, led_state.caps_lock); | ||
} | ||
return true; | ||
} |
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.
const rgblight_segment_t PROGMEM my_capslock_layer[] = RGBLIGHT_LAYER_SEGMENTS( | |
{0,1, HSV_WHITE} | |
); | |
const rgblight_segment_t* const PROGMEM my_rgb_layers[] = RGBLIGHT_LAYERS_LIST( | |
my_capslock_layer | |
); | |
void keyboard_post_init_kb(void) { | |
// Enable the LED layers | |
rgblight_layers = my_rgb_layers; | |
keyboard_post_init_user(); | |
} | |
bool led_update_kb(led_t led_state) { | |
if (led_update_user(led_state)) { | |
rgblight_set_layer_state(0, led_state.caps_lock); | |
} | |
return true; | |
} | |
#ifdef RGBLIGHT_LAYERS | |
const rgblight_segment_t PROGMEM my_capslock_layer[] = RGBLIGHT_LAYER_SEGMENTS( | |
{0,1, HSV_WHITE} | |
); | |
const rgblight_segment_t* const PROGMEM my_rgb_layers[] = RGBLIGHT_LAYERS_LIST( | |
my_capslock_layer | |
); | |
void keyboard_post_init_kb(void) { | |
// Enable the LED layers | |
rgblight_layers = my_rgb_layers; | |
keyboard_post_init_user(); | |
} | |
bool led_update_kb(led_t led_state) { | |
if (led_update_user(led_state)) { | |
rgblight_set_layer_state(0, led_state.caps_lock); | |
} | |
return true; | |
} | |
#endif |
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 file can be deleted.
), | ||
[1] = LAYOUT_all( | ||
_______, KC_F1, KC_F2, KC_F3, KC_F4, KC_F5, KC_F6, KC_F7, KC_F8, KC_F9, KC_F10, KC_F11, KC_F12, _______, _______, | ||
_______, RGB_TOG, RGB_MOD, RGB_HUI, RGB_HUD, RGB_SAI, RGB_SAD, RGB_VAI, RGB_VAD, _______, _______, _______, _______, _______, |
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.
_______, RGB_TOG, RGB_MOD, RGB_HUI, RGB_HUD, RGB_SAI, RGB_SAD, RGB_VAI, RGB_VAD, _______, _______, _______, _______, _______, | |
_______, UG_TOGG, UG_NEXT, UG_HUEU, UG_HUED, UG_SATU, UG_SATD, UG_VALU, UG_VALD, _______, _______, _______, _______, _______, |
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.
Still outstanding, and this file should also be renamed to keyboard.json
.
Thank you for your contribution! |
Thank you for your contribution! |
Description
Graystudio BD60 PCB Firmware
Types of Changes
Checklist