Skip to content

Allow changing tailwindcss colors by using css variables - #233

Open
zvikasdongre wants to merge 2 commits into
BlueprintFramework:mainfrom
zvikasdongre:tailwind-variables
Open

Allow changing tailwindcss colors by using css variables#233
zvikasdongre wants to merge 2 commits into
BlueprintFramework:mainfrom
zvikasdongre:tailwind-variables

Conversation

@zvikasdongre

Copy link
Copy Markdown
Contributor

This also does not have the previously occuring bug of graphs going black, and seems to work well with styled components as well.

I had to update tailwindcss to latest v3 version and twin.macro too, however this does not have any downsides or negative effects.

@prplwtf

prplwtf commented Sep 1, 2026

Copy link
Copy Markdown
Member

Looks good, I'll have to do some extensive testing to see if this actually works, I don't want to have what happened last time happen again 😭

@prplwtf
prplwtf self-requested a review September 1, 2026 13:57
@prplwtf prplwtf added the scope:frontend Issues or pull requests related to Blueprint's frontend. label Sep 1, 2026
@prplwtf
prplwtf requested a review from a team September 1, 2026 13:57
@zvikasdongre

zvikasdongre commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

OH, I completely forgot to mention this crucial step, clean the node_modules/.cache before rebuilding:

rm -rf node_modules/.cache

twin.macro uses this to cache tailwindcss color values, without clearing it, the colors in tw styles stay unaffected.

@prplwtf

prplwtf commented Sep 1, 2026

Copy link
Copy Markdown
Member

@0x7d8 Will clearing all the node_module cache screw up the build speed improvements we currently have.

@zvikasdongre If yes, is there also a way to specifically delete twin.macro cache?

@zvikasdongre

Copy link
Copy Markdown
Contributor Author

It only needs to be cleared once.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:frontend Issues or pull requests related to Blueprint's frontend.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants