There's an m_Pause variable and an IsGamePaused() function, which - as far as I could see - pretty much do the exact same thing, and could not find a reason why they were split like this. I played a few games with it and did not find any problems - I hope I did not miss anything

Currently you have to set "custom_colors": "true" in the skin json files. It would be far more reasonable and user friendly to remove this flag and automacally set it if Hue, Sat or lgt, or alpha is set.