-
-
Notifications
You must be signed in to change notification settings - Fork 39.8k
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
Add ability to blink lighting layer for a specified duration #8760
Conversation
Could you pul out the user code, here? Otherwise, LGTM. |
This reverts commit 831649b.
Done. |
Gentle ping... |
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.
Could you please run qmk cformat quantum/rgblight.c quantum/rgblight.h
and commit the changes it gives?
Other than that and the notes below, looks good to me.
Co-authored-by: James Young <[email protected]>
Co-authored-by: James Young <[email protected]>
Co-authored-by: James Young <[email protected]>
Done. |
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.
Looks good to me.
* Implement momentarily blink of lighting layers * Refactor spidey3 userspace to use rgb layer blink * Remove un-necessary line from example in documentation * Revert "Refactor spidey3 userspace to use rgb layer blink" This reverts commit 831649b. * Adds a missing bit of documentation about lighting layer blink * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * cformat, as suggested Co-authored-by: James Young <[email protected]>
* Implement momentarily blink of lighting layers * Refactor spidey3 userspace to use rgb layer blink * Remove un-necessary line from example in documentation * Revert "Refactor spidey3 userspace to use rgb layer blink" This reverts commit 831649b. * Adds a missing bit of documentation about lighting layer blink * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * cformat, as suggested Co-authored-by: James Young <[email protected]>
* Implement momentarily blink of lighting layers * Refactor spidey3 userspace to use rgb layer blink * Remove un-necessary line from example in documentation * Revert "Refactor spidey3 userspace to use rgb layer blink" This reverts commit 831649b. * Adds a missing bit of documentation about lighting layer blink * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * cformat, as suggested Co-authored-by: James Young <[email protected]>
* Implement momentarily blink of lighting layers * Refactor spidey3 userspace to use rgb layer blink * Remove un-necessary line from example in documentation * Revert "Refactor spidey3 userspace to use rgb layer blink" This reverts commit 831649b. * Adds a missing bit of documentation about lighting layer blink * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * cformat, as suggested Co-authored-by: James Young <[email protected]>
* Implement momentarily blink of lighting layers * Refactor spidey3 userspace to use rgb layer blink * Remove un-necessary line from example in documentation * Revert "Refactor spidey3 userspace to use rgb layer blink" This reverts commit 831649b. * Adds a missing bit of documentation about lighting layer blink * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * cformat, as suggested Co-authored-by: James Young <[email protected]>
* Implement momentarily blink of lighting layers * Refactor spidey3 userspace to use rgb layer blink * Remove un-necessary line from example in documentation * Revert "Refactor spidey3 userspace to use rgb layer blink" This reverts commit 831649b. * Adds a missing bit of documentation about lighting layer blink * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * cformat, as suggested Co-authored-by: James Young <[email protected]>
* Implement momentarily blink of lighting layers * Refactor spidey3 userspace to use rgb layer blink * Remove un-necessary line from example in documentation * Revert "Refactor spidey3 userspace to use rgb layer blink" This reverts commit 831649b. * Adds a missing bit of documentation about lighting layer blink * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * cformat, as suggested Co-authored-by: James Young <[email protected]>
* Implement momentarily blink of lighting layers * Refactor spidey3 userspace to use rgb layer blink * Remove un-necessary line from example in documentation * Revert "Refactor spidey3 userspace to use rgb layer blink" This reverts commit 831649b. * Adds a missing bit of documentation about lighting layer blink * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * Update docs/feature_rgblight.md per suggestions Co-authored-by: James Young <[email protected]> * cformat, as suggested Co-authored-by: James Young <[email protected]>
Description
This change adds optional functionality to allow blinking an RGB Lighting Layer on for a specified number of milliseconds. The purpose is to make it easy to use a lighting layer to acknowledge events (e.g. a settings change).
To use this new code, it must be enabled using
#define RGBLIGHT_LAYER_BLINK
e.g. in your config.h.Kudos to @n8gray for the great RGBLight layers implementation. It is my hope that this small addition will make it even more useful!
Types of Changes
Checklist