Skip to content
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

[WIP] Attempt hold-tap implementation of Magic key #2

Closed
wants to merge 7 commits into from

Conversation

chrisandreae
Copy link
Collaborator

Adds a hold-tap implementation of the magic key, which presently can't work because zmk,behavior-hold-tap demands two binding cells, and passes one each to its child bindings, while &rgb_ug itself requires two binding cells.

We'll have to work around this by adding a new &indicator behaviour that takes one argument, to be compatible with hold-tap.

moergo-sc and others added 6 commits July 29, 2022 18:20
Expects a keymap file as the POST body
Wrap compilation with ccache, and prepare a pre-warmed cache of the build in /tmp/ccache.
To maximize chance of a direct cache hit, change the lambda driver to always build in /tmp/build.

some back of the envelope measurements (2012 xeon e3-1230v2, nixos)
clean build, no cache -> 21.308
clean build, cache -> 7.145
modified keymap, clean build, cache -> 12.127
Does not yet include magic key hold-tap behaviour for the underglow status
@chrisandreae chrisandreae force-pushed the main branch 2 times, most recently from a68bae8 to a9417f3 Compare July 31, 2022 02:57
@chrisandreae chrisandreae deleted the keymap-development branch October 17, 2022 13:01
yuanbin pushed a commit to yuanbin/zmk that referenced this pull request Jun 14, 2023
* Add Contra Shield (moergo-sc#1)

* Add bluetooth control layer to contra keymap (moergo-sc#2)

* fix contra keymap issues

* add bluetooth control layer

* clean up contra files

* add contra.conf file

* add missing bracket for default_layer

* update copyright year to 2021

* add contra metadata file

* refactor pro micro overlay for new syntax
chrisandreae pushed a commit to chrisandreae/zmk that referenced this pull request Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants