I sometimes wonder if that code should be reworked - it has always seemed a little odd to me that we set the brightness when the lights are off (when we don't need to match any existing dash illumination levels) and then arbitrarily cut the illumination when the lights are on (when most people would probably want to match their dash levels).

It makes more sense to me to be easily able to set the lights-on button illumination level, and to have the lights-off setting relegated to config.ini.