-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
refactor(shields): MurphPad #1786
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.
I had noticed the discrepancies when comparing vs. the configurator, looks good to me.
61d7568
to
00eddca
Compare
00eddca
to
0b5302b
Compare
b5e8868
to
abda644
Compare
abda644
to
f944c84
Compare
f944c84
to
f75541c
Compare
f75541c
to
e563a67
Compare
b5b4fa0
to
e173060
Compare
d8b0e5e
to
639daad
Compare
639daad
to
5794c6f
Compare
5794c6f
to
a8ee00c
Compare
a8ee00c
to
a0706f8
Compare
571b5ce
to
6982f96
Compare
6982f96
to
187fd2a
Compare
187fd2a
to
207938e
Compare
* Added matrix transforms and physical layouts for three typical configurations. * Tweaked default keymap to be more immediately useful for ZMK and more closely resemble the default keymaps of non-ZMK firmware. * Board-specific Kconfig settings migrated to main shield file, so they will be exposed to end-users through the setup script. * Tidied formatting and shortened overlong layer labels. * Aligned encoder configuration with current standards. * Corrected default number of RGB LEDs. * Enabled OLED by default in a less authoritarian way.
207938e
to
7070c24
Compare
Looking at this the only point I want to raise is that it has OLED defaulting to |
I would agree with you on most shields. On this one: yes, you can opt out of the display, but the plate will have an unsightly hole in it unless you modify the plate file and get that alternate version cut. I've probably assisted with some fifty MurphPad builds in one way or another; plenty didn't bother with RGB or encoders, but no one has ever not installed the display...even if assembly errors meant they couldn't get it working. |
That's a fair argument. I think I'd still prefer explicit for consistency with other devices, though. We need to improve our related documentation on it too. I didn't actually notice that the encoder stuff wasn't commented out, I'd obviously be in favor of doing that too. All that said I am perfectly happy to be overruled in this case. |
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.
LGTM!
nice_nano
.Board/Shield Check-list