Jason
Jason
Duplicate of #914
See comment: > Would like to wait until just before the release to merge this. Or we can add a development branch for this repo.
**nitpick:** Can you group the `text-with` colors with the rest on lines 240 and 537? (as in, put the `.mainDarkGray` and `.secDarkGray` classes in those places, and remove lines 769...
To clarify, I'm referring to updating this section of the code:  And the other one corresponding to sec colors
You also need to set an explicit text color of black for the text color of light gray by adding the class to this selector:  and the secondary color...
@fdarcey Thanks for adding the light gray changes! Could you also move these `text-with-main-color` and `text-with-accent-color` parts grouped with the other classes in the comma-delimited selectors that I mention [here](https://github.com/FreeTubeApp/FreeTube/pull/4293#issuecomment-1830737334)...
@fdarcey Oh yes, to clarify, didn't mean to say that you didn't have the code present. I'm just saying to do it in that manner I proposed, because doing it...
Thanks for your patience @fdarcey. Final asks: Please just add `.mainLightGray` and `.mainDarkGray` to the top two sections instead of giving them their own sections. The reason is that code...
Hi @fdarcey, I apologize for the communication troubles. I've created [this PR here](https://github.com/fdarcey/FreeTube/pull/1) with a commit for implementing the suggested changes (& resolving merge conflicts).
To clarify, the above issues detected are merge conflict issues not detected by the GH action, and these are all resolved in my commit above.