Skip to content

fix: stop shellui-theme.css requesting tailwindcss when linked directly - #44

Merged
Shewart merged 1 commit into
mainfrom
fix/theme-css-import
Oct 10, 2026
Merged

Shewart merged 1 commit into
mainfrom
fix/theme-css-import

Conversation

@Shewart

@Shewart Shewart commented Oct 10, 2026

Copy link
Copy Markdown
Contributor

src/ShellUI.Components/wwwroot/shellui-theme.css started with @import "tailwindcss";. A page that links the theme directly (<link href="_content/ShellUI.Components/shellui-theme.css">, as the docs site does) made the browser request tailwindcss as a URL and log a 404 on every load.

  • shellui-theme.css now holds only the theme
  • scripts/rebuild-precompiled-css.sh writes @import "tailwindcss"; at the top of its generated Tailwind input; it is the only thing that builds from the theme file

Checked:

  • shellui-all.css is byte-for-byte identical before and after
  • A fresh Blazor app linking shellui-theme.css from the package: the stylesheet loads, there is no tailwindcss request, the console is clean, and --background applies
  • All 303 tests pass

The theme started with @import "tailwindcss";, so a page that links it
asked the browser for a tailwindcss URL and logged a 404 on every load.
The CSS build script now writes the import into its generated input, so
the theme file holds only the theme and shellui-all.css is unchanged.
Copilot AI balanced review requested due to automatic review settings October 10, 2026 13:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Shewart
Shewart merged commit 16d18c3 into main Oct 10, 2026
1 check passed
@Shewart
Shewart deleted the fix/theme-css-import branch October 10, 2026 14:04
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