nerves-keyboard / xebow

Firmware for the Keybow written in Elixir
40 stars 10 forks source link

Move "configurables" from Engine to Xebow #126

Closed vanvoljg closed 4 years ago

vanvoljg commented 4 years ago

This aims to clean up the Engine a bit more by moving the configurables into Xebow. Because Xebow is the general place where configuration happens, it seemed the most logical place to put it for now.

Configurables seem... strange. They only get used in the Live View to make sure the display elements are consistent with the current configuration state. I want there to be a better way to push those updates down to the Live View, but I haven't taken a closer look at it, yet.

Anyway, because I had to work in the Xebow section, I added a good number of tests to it. Tried to cover almost everything.

Updated a bit of typing and documentation in Animation and Config, but it's minor.