Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Webcomponents v4 - les feuilles de style ont un impact sur l'entièreté de la page #932

Open
vbombaerts opened this issue Oct 15, 2024 · 2 comments
Assignees

Comments

@vbombaerts
Copy link

A propos de
Inventaire des données et cartographie - L'Environnement en Wallonie

Un commentaire purement technique. C’est un peu dommage que vous incluiez des feuilles de style qui redéfinissent des balises générales (Body, HTML, H1, P…) . Ce n’est pas un bonne pratique car ça interfère avec la feuille du style du site dans laquelle on insère vos scripts. Vos feuilles de style ne devraient contenir que des classes exclusivement utilisées pour vos besoin. A voir pour l’avenir si c’est appelé à être utilisé plus largement.

Tu peux comparer avec une page normale du portail pour constater que ta CSS modifie complètement le look du site
Données vis-à-vis de l'Europe - L'Environnement en Wallonie

@fxprunayre
Copy link

fxprunayre commented Oct 16, 2024

Les webcomposants v4 sont pour le moment sans shadowDOM, donc inclus dans la page (avec leur style).

image

Il faudrait les passer en mode shadowDOM pour qu'ils soient isolés (comme les composants v5).

image

On peut activer le mode shadowDOM avec la librairie utilisée react-to-webcomponent mais je ne vois pas pour injecter les styles dans le composant.

Même problème dans #924

@vbombaerts
Copy link
Author

Ca peut attendre la v5

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

No branches or pull requests

3 participants