Skip to content

Conversation

@natemoo-re
Copy link
Member

This PR makes pnpm dev-ui and Vercel preview deployments reflect user theme preferences

By inlining a small script to the index.ejs template, we can avoid Flash of inAccurate coloR Theme (aka FART). Unfortunately, the theme is asynchronously fetched, but by defaulting to body.theme-system, the likelihood of an unstyled flash should be greatly reduced.

@natemoo-re natemoo-re requested review from a team and evanpurkhiser October 23, 2025 21:22
Copy link
Member

@JonasBa JonasBa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need the same removal on the if (theme) line, else lgtm, thanks for improving this!

@natemoo-re natemoo-re enabled auto-merge (squash) October 24, 2025 17:29
@natemoo-re natemoo-re merged commit a293eee into master Oct 24, 2025
42 checks passed
@natemoo-re natemoo-re deleted the feat/spa-theme branch October 24, 2025 17:36
natemoo-re added a commit that referenced this pull request Oct 24, 2025
I accidentally merged some broken response logic in #102045.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants