Skip to content

Boutons absent en raison de BOM dans le css minifié : bump de version de css-loader - #5616

Merged
vdegove merged 4 commits into
masterfrom
5614-fix-buttons
Sep 7, 2026
Merged

vdegove merged 4 commits into
masterfrom
5614-fix-buttons

Conversation

@vdegove

@vdegove vdegove commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Il y avait un truc comme ça dans le fichier minifié (un BOM ) :

...{margin:0}<EF BB BF>:root{--primary-color-producer:var(--theme-primary);--primary-color-reuser:#287233}

En raison d’updates sur FontAwesome + PostCSS + CSSLoader.

Voir issue upstream webpack/css-loader#1678 ça a été réglé dans une nouvelle version de CSSLoader : webpack/css-loader#1684

Du coup j’ai juste bump la version de CSSLoader. C’est une version un peu trop récente à mon goût (28 août 2026) mais qui ne contient que ce fix là, et qui semble ok https://github.com/webpack/css-loader/releases/tag/v7.1.5

Testé sur prochainement, les boutons sont revenus.

vdegove and others added 3 commits September 7, 2026 16:03
This reverts commit 5c2dbb0. Upstream fixed the mid-file BOM in css-loader 7.1.5, so bump it instead of forcing sassOptions.charset to false.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@socket-security

socket-security Bot commented Sep 7, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedcss-loader@​7.1.4 ⏵ 7.1.59910010084 -4100

View full report

@vdegove vdegove changed the title Boutons absent : BOM dans le css minifié Boutons absent en raison de BOM dans le css minifié : bump de version de css-loader Sep 7, 2026
@vdegove
vdegove marked this pull request as ready for review September 7, 2026 14:28
@vdegove
vdegove requested a review from a team as a code owner September 7, 2026 14:28
@vdegove
vdegove enabled auto-merge September 7, 2026 14:42

@ptitfred ptitfred 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.

Crazy!

@vdegove
vdegove added this pull request to the merge queue Sep 7, 2026
Merged via the queue into master with commit ff090b8 Sep 7, 2026
5 checks passed
@vdegove
vdegove deleted the 5614-fix-buttons branch September 7, 2026 15:02
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