Color Schema not working correctly

[ModdyLP]

Hay,

i dont know if it my fault, but after i updated the theme to the newest version, my color scheme is ignored. I looked into the compiled main.css and saw that there are some changes of colors from a variable to a static color. Maybe this is conflicting with the schema variables.

Can someone test this and verify if this is a bug or my fault? Maybe I updated the files wrong.

[ModdyLP]

After this Commit: 4cefe3a the Rebuild causes the break maybe: 5b9e285

[JVimes]

To help anyone in the meantime, here's a /generated/main.css built from latest Blowfish (with old tw-elements) that seems to work.

[nunocoracao]

@ModdyLP @JVimes I didn't have time to look into this but I am running thus in Blowfish and my personal website with no issues. I wouldn't advise replacing that css file as it might cause future problems. Can I get more details on the problem, potentially a repo URL?

[ModdyLP]

@nunocoracao Sure, here is my repo with a branch where I build the update and pushed it. Simple start the example and then you will see that the fire schema is not applied and the compiled main.css looks wrong. https://github.com/ModdyLP/blowfish/tree/fix_Colors

I am also investigating and looking for a fix, i teared it down to an update from tailwindcss in tailwind elements from version 3.0.7 to 3.2.4. Between these versions many fixes are done and the parsing was extended to support rgb and hsl codes. Maybe the parser is not working correctly. As i mentioned there is already a issue, that the styles vars do not apply if there is a alpha notation. I tried removing it and some other things, sometimes it works partly. I will try to fix it and provide it if i found something.

[nunocoracao]

OK found the problem

[nunocoracao]

issuing a fix soon
#611

[JVimes]

Me too. I don't think I see anything reading --color-primary- or --color-secondary colors other than chart.js. The button background color seems to be hard coded.

[nunocoracao]

fixed in #611

[ModdyLP]

Very Nice its working now, is this require not necessary anymore?

[nunocoracao]

never was... leftovers of when I was trying to figure out the breaking changes from tw alpha to beta. Tested the shortcodes that use it and everything seems to be ok.

[ModdyLP]

Ah, okay understandable. That was a line i never thought about. Because I thought it was necessary and didn't touch it.