-
-
Notifications
You must be signed in to change notification settings - Fork 39.6k
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
[Core] Add Layer Lock feature #23430
Conversation
This is great! Thank you drashna for making this PR to add Layer Lock in QMK core. Please let me know if I can be of help. |
welcome! and mostly, I think I just need to write docs for this. The main issue being where to add them. |
d8a4ddb
to
8370f4c
Compare
This commit builds on @drashna's qmk#23430 to add documentation: * New page docs/features/layer_lock.md added describing Layer Lock. * Added link to this page in docs/_sidebar.json * And in docs/keycodes.md * And a cross-reference in docs/feature_layers.md NOTE: Beware, I haven't checked this documentation locally with `qmk docs` (the command failed with a problem involving yarn). So there might be formatting bugs and broken links in this.
Nit: "I have updated the documentation accordingly" in the PR checklist can now be checked 👍 |
This is a really cool feature, excited to see it incorporated into main! |
because drashna never makes spelling mistakes! Co-authored-by: Daniel <1767914+iamdanielv@users.noreply.github.com>
Co-authored-by: Daniel <1767914+iamdanielv@users.noreply.github.com> Co-authored-by: Pascal Getreuer <getreuer@google.com> Co-authored-by: Pascal Getreuer <50221757+getreuer@users.noreply.github.com>
Description
Types of Changes
Issues Fixed or Closed by This PR
Checklist