Another thing is, wouldn't it be more efficient to change the colors in Colors.tsx instead of passing the settings to all components?
The main issue with this approach is backwards compatibility. Everyone is using default exports "Colors" as a constant. Changing that would require changing all other code that deal with Colors. I preferred passing the option and making the feature optional for now rather than clashing with existing code. Once things are more stable, it can be changed.
I updated the branch the bug should be resolved now
The main issue with this approach is backwards compatibility. Everyone is using default exports "Colors" as a constant. Changing that would require changing all other code that deal with Colors. I preferred passing the option and making the feature optional for now rather than clashing with existing code. Once things are more stable, it can be changed.
I updated the branch the bug should be resolved now