Don't enable LED on Key_NoKey

[tchernomax]

Currently, when numpad layer is on, LED are enabled on XXX keys.
I think it's not what we want.

[algernon]

At a first read, this looks good. There is maybe one corner case I'd test before merging it: if we have three layers active, the base one, a second one and NumPad, and the second one has XXXs, which the active LEDMode does highlight/animate, will this change stop that?

It probably doesn't, but the interactions between LEDModes and overriding plugins like this is complicated enough that I don't trust my first impressions. :)

[tchernomax]

@algernon did you mean the following ?

enum { QWERTY, FUNCTION, NUMPAD };
…
const Key keymaps[][ROWS][COLS] PROGMEM = {
[QWERTY] = KEYMAP_STACKED( …, KEY_Q, …),
[FUNCTION] = KEYMAP_STACKED( …, XXX, …),
[NUMPAD] = KEYMAP_STACKED( …, ___, …)}
…

With this configuration, when I press numlock, Q isn't lighten up ; with or without the patch.

[algernon]

Yep, that's what I meant. Thanks for testing!