keyboardio / Chrysalis

Graphical configurator for Kaleidoscope-powered keyboards
https://github.com/keyboardio/Chrysalis#chrysalis
GNU General Public License v3.0
496 stars 64 forks source link

Editor: Layer copy & paste should not depend on Colormap #1172

Closed algernon closed 2 years ago

algernon commented 2 years ago

It is not required for a device to have any colormap-related functionality, whether it has LEDs or not. Copying and pasting layers should function regardless of the presence of colormaps.

As such, make copyLayer() handle the case of a missing colormap correctly, and do not require colormap for hasCopiedLayer().

With these fixes, it is now possible to copy & paste layers on an Atreus.

Originally reported on the forums.