Skip to content

Fix styles wtih BOM ignored by browsers, leading to UI behavior change - #5544

Merged
boutell merged 1 commit into
mainfrom
incident-postcss-bom
Aug 4, 2026
Merged

Fix styles wtih BOM ignored by browsers, leading to UI behavior change#5544
boutell merged 1 commit into
mainfrom
incident-postcss-bom

Conversation

@myovchev

@myovchev myovchev commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Please indicate which branch this PR should merge into:

Check one

  • main
  • latest
  • stable
  • Check if this PR will be resubmitted against another branch

Summary

Fixed incorrect sizing and spacing across the admin UI in webpack builds. A byte order mark preserved by PostCSS 8.5.24 invalidated the stylesheet rule setting box-sizing for .apos- elements. Sass is now compiled with charset: false in webpack builds, so the marker is never emitted. Vite builds were unaffected.

What are the specific steps to test this change?

No UI visual change in projects with Webpack builds.

What kind of change does this PR introduce?

(Check at least one)

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Build-related changes
  • Other

Make sure the PR fulfills these requirements:

  • It includes a) the existing issue ID being resolved, b) a convincing reason for adding this feature, or c) a clear description of the bug it resolves
  • The changelog is updated
  • Related documentation has been updated
  • Related tests have been updated

If adding a new feature without an already open issue, it's best to open a feature request issue first and wait for approval before working on it.

Other information:

@myovchev
myovchev requested a review from boutell August 4, 2026 10:48
@boutell
boutell merged commit 4f01649 into main Aug 4, 2026
21 checks passed
@boutell
boutell deleted the incident-postcss-bom branch August 4, 2026 12:55
@boutell

boutell commented Aug 4, 2026

Copy link
Copy Markdown
Member

Merged so the next tests of other things pass. Thanks.

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