SPW-DIG / metawal-core-geonetwork

Metawal - Catalogue pour l'information géographique de Wallonie
http://metawal.wallonie.be
GNU General Public License v2.0
3 stars 1 forks source link

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

Open vbombaerts opened 4 days ago

vbombaerts commented 4 days ago

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 commented 4 days ago

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 https://github.com/SPW-DIG/metawal-core-geonetwork/issues/924

vbombaerts commented 4 days ago

Ca peut attendre la v5