Skip to content

Conversation

@Yannicvanveen
Copy link

Het wrappen van de editor styles zit al in het vite-config package (vandaar de update). Deze PR verwijdert de originele .editor-styles-wrapper uit editor.css en schakelt het wrappen van de editor styles in vanuit de vite config in vite.config.js.

@WybeBosch
Copy link
Contributor

Ik wilde ff checken waar in de vite-config de nieuwe logica geregeld werd dat we editor-styles-wrapper niet meer nodig hadden

image

maar kan de repo niet vinden? Is dit github search being an ass, of staat team wordpress nog niet toegevoegd aan die repo?

@Yannicvanveen
Copy link
Author

Ik wilde ff checken waar in de vite-config de nieuwe logica geregeld werd dat we editor-styles-wrapper niet meer nodig hadden

image maar kan de repo niet vinden? Is dit github search being an ass, of staat team wordpress nog niet toegevoegd aan die repo?

Ik geloof dat die onderdeel is van de toolkit monorepo.

@WybeBosch
Copy link
Contributor

WybeBosch commented Jan 8, 2026

Ik wilde ff checken waar in de vite-config de nieuwe logica geregeld werd dat we editor-styles-wrapper niet meer nodig hadden

maar kan de repo niet vinden? Is dit github search being an ass, of staat team wordpress nog niet toegevoegd aan die repo?

Ik geloof dat die onderdeel is van de toolkit monorepo.

Ahh right inderdaad danku, gevonden :)
Hier staat nu de logica dat vite zelf de wrapper toevoegt
https://github.com/yardinternet/toolkit/blob/main/packages/vite-config/src/utils/get-postcss-prefixwrap-plugin.js#L14

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.

3 participants