Use the new Layer.lookupOnActiveLayer function

We want to highlight keys as specified by the layer, not as their current role,
otherwise we may be missing keys we should highlight.

Thanks to @cdisselkoen for the report and the reproduction steps! Fixes #3.

Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
pull/389/head
Gergely Nagy 7 years ago
parent 453e30038f
commit 2e1ae36cff

@ -69,7 +69,7 @@ void ActiveModColorEffect::loopHook(bool is_post_clear) {
for (byte r = 0; r < ROWS; r++) { for (byte r = 0; r < ROWS; r++) {
for (byte c = 0; c < COLS; c++) { for (byte c = 0; c < COLS; c++) {
Key k = Layer.lookup(r, c); Key k = Layer.lookupOnActiveLayer(r, c);
if (isModifierActive(k) || isLayerKeyActive(k)) if (isModifierActive(k) || isLayerKeyActive(k))
LEDControl.setCrgbAt(r, c, highlight_color); LEDControl.setCrgbAt(r, c, highlight_color);

Loading…
Cancel
Save