-
-
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
docs: move layer defines to reduce confusion #2639
Conversation
Moved the section about defines to the bottom, clearifing layer behaviors take a number as argument.
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.
Thanks for remembering and following up! Good change, added one suggestion to improve.
Co-authored-by: Cem Aksoylar <[email protected]>
Considering the rephrasing of the second sentence, can we link to sticky |
There is already some inconsistency regarding defining layer-tap here but not sticky layer, so ideally we'd do something similar where we define sticky layer here too. Also see #2273. I think the aforementioned can be done in the PR that'd handle #2273, for now maybe just a note like conditional layers would be great. As it tends to go: the more/better work you do, the more work we expect 😄 |
We'll handle above in a separate PR. |
…johanson/zmk into petejohanson-feat/pointers-with-input-processors * 'feat/pointers-with-input-processors' of github.com:petejohanson/zmk: (186 commits) docs: Add initial pointer docs. feat: Add input split support. feat(mouse): Add mouse move and scroll support feat(boards): Update for mikoto board definition (zmkfirmware#1946) refactor(Kconfig): Extracted designer defaults out into new files (zmkfirmware#2537) chore(main): release 0.1.0 (zmkfirmware#2657) docs: Create a Hardware Integration index page (zmkfirmware#2634) docs: Mention combos in reset behaviors (zmkfirmware#2677) fix(boards): Disable uart serial node in Xiao BLE by default (zmkfirmware#2672) chore(deps): bump the prod-other-minor-patch group across 1 directory with 2 updates feat(boards): Add glove80 nexus node for extension GPIO. (zmkfirmware#2594) docs: Move defines to the end in layer behaviors (zmkfirmware#2639) docs: Update new-shield.mdx (zmkfirmware#2664) docs(ci): Netlify ignore command to check branch (zmkfirmware#2659) chore(deps): bump cross-spawn from 7.0.3 to 7.0.6 in /docs feat(ci): Add release-please automation with VERSION (zmkfirmware#2622) docs: Remove dangling 0 in sticky keys docs fix: include a header file for RC macros (zmkfirmware#2649) feat(drivers): Support init high/low in 595 driver docs: touchups on the soft off information for improved clarity. (zmkfirmware#2647) ...
…fingerpunch * petejohanson-feat/pointers-with-input-processors: (186 commits) docs: Add initial pointer docs. feat: Add input split support. feat(mouse): Add mouse move and scroll support feat(boards): Update for mikoto board definition (zmkfirmware#1946) refactor(Kconfig): Extracted designer defaults out into new files (zmkfirmware#2537) chore(main): release 0.1.0 (zmkfirmware#2657) docs: Create a Hardware Integration index page (zmkfirmware#2634) docs: Mention combos in reset behaviors (zmkfirmware#2677) fix(boards): Disable uart serial node in Xiao BLE by default (zmkfirmware#2672) chore(deps): bump the prod-other-minor-patch group across 1 directory with 2 updates feat(boards): Add glove80 nexus node for extension GPIO. (zmkfirmware#2594) docs: Move defines to the end in layer behaviors (zmkfirmware#2639) docs: Update new-shield.mdx (zmkfirmware#2664) docs(ci): Netlify ignore command to check branch (zmkfirmware#2659) chore(deps): bump cross-spawn from 7.0.3 to 7.0.6 in /docs feat(ci): Add release-please automation with VERSION (zmkfirmware#2622) docs: Remove dangling 0 in sticky keys docs fix: include a header file for RC macros (zmkfirmware#2649) feat(drivers): Support init high/low in 595 driver docs: touchups on the soft off information for improved clarity. (zmkfirmware#2647) ...
Moved the section about defines to the bottom, clearifing layer behaviors take a number as argument.