[SPIP Zone] [Spip-zone-commit] r109601 - in _plugins_/oembed

Archives Zone
1

Hello Placido,

Merci de tes corrections !

Toutefois dans ce genre de modification il faut toujours faire attention à ce que ça ne casse pas le rendu pour ceux qui auraient surchargés le modèle chez eux pour y faire une petite modif et auront donc l’ancien markup. Peut-être ça peut marcher si on combine le text-indent d’une part et le span de l’autre ?

A contrario on essaye aussi de faire que ça ne casse rien si les utilisateurs ont surchargé oembed.css pour y faire une petite modif de style et se retrouvent donc avec le nouveau markup mais sans les styles sur les éléments que tu as a jouté.
C’est par exemple la raison de l’utilisation du <br />. Il n’est pas formidable mais il peut aussi se masquer complètement en CSS si tu en as envie.

Sinon on gère pas ces cas là, ça veut dire que ce n’est pas un upgrade mineur car les sites utilisant le plugin risquent d’avoir des rendus cassés.

Par ailleurs utiliser un span.label est une mauvaise idée, car le nommage label est trop répandu, dans BootStrap par exemple il va hériter de tout un tas de style qui ne seront pas du tout appropriés.

Toutes les classes utilisées par le modèle sont préfixées par oe- et c’est pour éviter ce genre de collision.

--
Cédric

On 18 mars 2018 à 19:35 +0100, spip-zone-commit@rezo.net, wrote:

Author: placido@roxing.net
Date: 2018-03-18 19:34:50 +0100 (Sun, 18 Mar 2018)
New Revision: 109601

Modified:
_plugins_/oembed/css/oembed.css
_plugins_/oembed/modeles/oembed_photo.html
_plugins_/oembed/modeles/oembed_rich.html
_plugins_/oembed/modeles/oembed_video.html
_plugins_/oembed/paquet.xml
Log:
Compléter le markup du modèle, pour cibler plus facilement certains éléments
- label du bouton play dans un <span/> pour le cacher mais le laisser accessible. L'astuce précédente avec text-indent avait le désavantage de générer un outline pas très heureux lors du focus (cf: https://pic.infini.fr/WWWPT6M7/nRzupz1N.png).
- Harmoniser :focus et :hover
- Remplacer le <br> par du style CSS

Details: Connexion · GitLab

_______________________________________________
Spip-zone-commit@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone-commit

2

Salut Cédric,

Oui, j'ai bien conscience qu'on marche sur des oeufs de ne pas casser ce
qui est déjà en place.

En dépit de mes efforts dans ce sens, as-tu constater de la casse après
upgrade ?

Peut-être ça peut marcher si on combine le text-indent d’une part et le span de l’autre ?

OK, pour rétablir le text-indent en parallèle du span. Ça n'annule pas
la correction et préservera le comportement de l'ancien markup. (désolé
pour l'oubli).

Pour le <br/>, normalement disaplay:inline-block et width:100% sur le
parent amène au même résultat. Mais si tu penses qu'il faut le rétablir,
je le fais (et continuerais à le masquer de mon côté).

Pour les class, que proposes-tu :

préfixe sytématique ? span.oe-label , span.oe-par, .oe-nom (ou .oe-name)