Skip to content

Docs: drop the purple theme-color now that BUI sets one per color scheme - #43347

Merged
julien-deramond merged 1 commit into
mainfrom
v6-jd-remove-theme-color
Nov 11, 2026
Merged

julien-deramond merged 1 commit into
mainfrom
v6-jd-remove-theme-color

Conversation

@julien-deramond

@julien-deramond julien-deramond commented Nov 11, 2026 •

Copy link
Copy Markdown
Member

Since @twbs/bui 0.1.10 (#43339), BUI's <Head> outputs a theme-color per color scheme (#fff for light, #080a0c for dark, matching v6's body background) before the rest of the head. Browsers use the first theme-color that matches, so our own <meta name="theme-color" content="#712cf9"> in Favicons.astro comes after them and is never used. This removes it, so each page has only the per-scheme pair. It depends on #43339, and before that one is merged the line is still the site's only theme-color.

The purple brand color is kept where it still means something: the mask-icon color and the web app manifest's theme_color are unchanged.

On the deploy preview of #43339, /docs/getting-started/introduction/ had four tags in this order:

<meta name="color-scheme" content="light dark">
<meta name="theme-color" content="#fff" media="(prefers-color-scheme: light)">
<meta name="theme-color" content="#080a0c" media="(prefers-color-scheme: dark)">
<meta name="theme-color" content="#712cf9">

After this change, the page should have the first three only. site/src/assets/stackblitz.js still strips meta[name="theme-color"] from StackBlitz examples, which now removes BUI's pair instead.

curl -s https://deploy-preview-43347--twbs-bootstrap.netlify.app/docs/getting-started/introduction/ \
  | grep -oE '<meta name="(theme-color|color-scheme)"[^>]*>'

Results:

<meta name="color-scheme" content="light dark">
<meta name="theme-color" content="#fff" media="(prefers-color-scheme: light)">
<meta name="theme-color" content="#080a0c" media="(prefers-color-scheme: dark)">

🤖 Generated with Claude Code

@netlify

netlify Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for twbs-bootstrap ready!

Name Link
🔨 Latest commit 2dc334b
🔍 Latest deploy log https://app.netlify.com/projects/twbs-bootstrap/deploys/6af438cb563b7dd20dbe773f
😎 Deploy Preview https://deploy-preview-43347--twbs-bootstrap.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

…cheme

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@julien-deramond
julien-deramond force-pushed the v6-jd-remove-theme-color branch from 83ea71e to 2dc334b Compare November 11, 2026 09:43
@julien-deramond julien-deramond added docs Documentation site and its content v6 v6, the main branch labels Nov 11, 2026
@github-project-automation github-project-automation Bot moved this to Inbox in v6.0.0 Nov 11, 2026
@julien-deramond julien-deramond added this to the v6.0.0-alpha.2 milestone Nov 11, 2026
@julien-deramond
julien-deramond merged commit 795268b into main Nov 11, 2026
14 checks passed
@julien-deramond
julien-deramond deleted the v6-jd-remove-theme-color branch November 11, 2026 09:47
@coliff

coliff commented Nov 12, 2026

Copy link
Copy Markdown
Contributor

@twbs/bui looks really interesting! (https://www.npmjs.com/package/@twbs/bui).
Will it be open-sourced? The GitHub repo is private right now...

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

Labels

docs Documentation site and its content v6 v6, the main branch

Projects

Status: Inbox

Development

Successfully merging this pull request may close these issues.

2 participants