spip-zone-commit@rezo.net a écrit le 06/02/2017 à 00:03 :
Author: tetue@rezo.net
Date: 2017-02-06 00:03:52 +0100 (Mon, 06 Feb 2017)
New Revision: 102664
Modified:
_plugins_/figure/modeles/doc_legende.html
_plugins_/figure/modeles/image.html
_plugins_/figure/modeles/img.html
_plugins_/figure/plugin.xml
Log:
Utiliser le bon filtre sur les attributs
Il me semblait que le title était à mettre sur le lien, et le alt seulement sur l'image dans
[<a href="(#ENV{lien})"[ class="(#ENV{lien_class})"]>]<img src='#URL_DOCUMENT' alt="[(#TITRE|attribut_html)]"[ title="(#TITRE|attribut_html)"][ width="(#LARGEUR)"][ height="(#HAUTEUR)"]>[(#ENV{lien}|?{</a>})]
Le 6 févr. 2017 à 00:07, RealET <real3t@gmail.com> a écrit :
spip-zone-commit@rezo.net a écrit le 06/02/2017 à 00:03 :
Author: tetue@rezo.net
Date: 2017-02-06 00:03:52 +0100 (Mon, 06 Feb 2017)
New Revision: 102664
Modified:
_plugins_/figure/modeles/doc_legende.html
_plugins_/figure/modeles/image.html
_plugins_/figure/modeles/img.html
_plugins_/figure/plugin.xml
Log:
Utiliser le bon filtre sur les attributs
Il me semblait que le title était à mettre sur le lien, et le alt seulement sur l'image dans
[<a href="(#ENV{lien})"[ class="(#ENV{lien_class})"]>]<img src='#URL_DOCUMENT' alt="[(#TITRE|attribut_html)]"[ title="(#TITRE|attribut_html)"][ width="(#LARGEUR)"][ height="(#HAUTEUR)"]>[(#ENV{lien}|?{</a>})]
Qu'en dis-tu ?
Dans le cas des images liens, lorsque l’attribut title est présent, il doit être strictement identique au contenu de l'attribut alt et également porté par img.
Oui, ça me perturbe aussi
Voir l’exemple (perdu parmi les autres) à cette page :
Le 6 févr. 2017 à 00:07, RealET <real3t@gmail.com> a écrit :
spip-zone-commit@rezo.net a écrit le 06/02/2017 à 00:03 :
Author: tetue@rezo.net
Date: 2017-02-06 00:03:52 +0100 (Mon, 06 Feb 2017)
New Revision: 102664
Modified:
_plugins_/figure/modeles/doc_legende.html
_plugins_/figure/modeles/image.html
_plugins_/figure/modeles/img.html
_plugins_/figure/plugin.xml
Log:
Utiliser le bon filtre sur les attributs
Il me semblait que le title était à mettre sur le lien, et le alt seulement sur l'image dans
[<a href="(#ENV{lien})"[ class="(#ENV{lien_class})"]>]<img src='#URL_DOCUMENT' alt="[(#TITRE|attribut_html)]"[ title="(#TITRE|attribut_html)"][ width="(#LARGEUR)"][ height="(#HAUTEUR)"]>[(#ENV{lien}|?{</a>})]
Qu'en dis-tu ?
Dans le cas des images liens, lorsque l’attribut title est présent, il doit être strictement identique au contenu de l'attribut alt et également porté par img.
Merci pour le plugin déjà ça manquais j'avais surchargé dans mon coin, mais autant utiliser un plugin.
Je remarque sur mes test que la class d'alignement n'est plus `[ spip_documents_(#ENV{align})]` mais `spip_documents[ (#ENV{align})]`
ce qui dans mon cas téléscope dans le corps des articles avec une class .center, est ce voulu ? une coquille ?
Merci pour le plugin déjà ça manquais j'avais surchargé dans mon coin, mais autant utiliser un plugin.
Je remarque sur mes test que la class d'alignement n'est plus `[ spip_documents_(#ENV{align})]` mais `spip_documents[ (#ENV{align})]`
ce qui dans mon cas téléscope dans le corps des articles avec une class .center, est ce voulu ? une coquille ?
Ah, oui mince, exact. J’ai sans doute fait ça un peu vite, présupposant que les class spip_documents_toto n’étaient plus utilisées, à l’heure des frameworks définissant des class génériques .left, .right et .center.
Juste pour bien comprendre, que fais ta class .center ?
Merci pour le plugin déjà ça manquais j'avais surchargé dans mon coin, mais autant utiliser un plugin.
Je remarque sur mes test que la class d'alignement n'est plus `[ spip_documents_(#ENV{align})]` mais `spip_documents[ (#ENV{align})]`
ce qui dans mon cas téléscope dans le corps des articles avec une class .center, est ce voulu ? une coquille ?
Ah, oui mince, exact. J’ai sans doute fait ça un peu vite, présupposant que les class spip_documents_toto n’étaient plus utilisées, à l’heure des frameworks définissant des class génériques .left, .right et .center.
Juste pour bien comprendre, que fais ta class .center ?
Alors pour l'histoire , sur un site asso les rédacs avaient introduit sur les conseil d'un Joomleu l'horrible balise <center>
quand j'ai vu ça a la mise a jour d'après (1 an) j'ai exprimé mon mécontentement
mais trop tard y'en avait partout dans près de 4000 articles, et dans les chapo, descriptifs, titres, même dans les raccourcis typo … bref la cata !
Gâce a spip, j'ai utilisé textWheel, pour remplacer a la volé la balise <center> par un div.center ,
et je me suis retrouvé avec cette class qui maintenant a vrai dire me fait un peut honte car trop générique je trouve, mais bon …
avec un .texte .center ça se maitrise a peut près si ce n'est pas défini en amont.
Et voila comment en testant figure, je m'en suis rendu compte.
Le plugin ne casse presque rien changeant surtout les balises HTML… MAIS il casse les titres/descriptifs en mettant la classe "spip_doc_descriptif" en *conteneur* de "spip_doc_titre", ce qui n'a aucun rapport.
spip_doc_titre sert à cibler le contenu du champ #TITRE et spip_doc_descriptif sert à cibler le champ #DESCRIPTIF, tout comme d'ailleurs normalement spip_doc_credits devrait servir à cibler le champ #CREDITS, c'est un manque.
Qu'il y ait besoin d'une classe englobante pour cibler le parent, alors là oui tout à fait ! Mais PAS spip_doc_descriptif, qui a toujours ciblé uniquement le champ éponyme, ce qui est logique, ça sert à ça.
Donc il faudrait trouver une autre classe pour le parent, et remettre spip_doc_descriptif uniquement autour de #DESCRIPTIF.
Par exemple on peut vouloir masquer en CSS tous les descriptifs, et ne laisser que les titres. Là désormais nos styles d'avant masque tout le bloc complet de texte.
Le plugin ne casse presque rien changeant surtout les balises HTML… MAIS il casse les titres/descriptifs en mettant la classe "spip_doc_descriptif" en *conteneur* de "spip_doc_titre", ce qui n'a aucun rapport.
Effectivement
spip_doc_titre sert à cibler le contenu du champ #TITRE et spip_doc_descriptif sert à cibler le champ #DESCRIPTIF, tout comme d'ailleurs normalement spip_doc_credits devrait servir à cibler le champ #CREDITS, c'est un manque.
Clair, d'ailleurs je ne sais pas si le choix de <small>, suivant la base css utilisé n'obligera pas a surcharger (par exemple si on est en 'em', on peut se retrouver trop petit en corps de caractère), à voir, mais spip_doc_credits me semble
Qu'il y ait besoin d'une classe englobante pour cibler le parent, alors là oui tout à fait ! Mais PAS spip_doc_descriptif, qui a toujours ciblé uniquement le champ éponyme, ce qui est logique, ça sert à ça.
Donc il faudrait trouver une autre classe pour le parent, et remettre spip_doc_descriptif uniquement autour de #DESCRIPTIF.
Par exemple on peut vouloir masquer en CSS tous les descriptifs, et ne laisser que les titres. Là désormais nos styles d'avant masque tout le bloc complet de texte.
dans le cas de figure > caption on cibler sur l'élément `figure.spip_document > caption`, même si ce n'est pas recommandé de cibler une balise on pourrait encore le tolérer dans ce cas…
Après quelques test en fait, cette modiifcation impacte sur le cas des documents
.center, du coup ça oblige a modifier les css de la dist, ou a ce que figure ajoute ses propres surcharges
ce qui je crois avoir compris n’était pas forcément la volonté de Tetue…
ça “me paraitrait” être un bon compromis et compat ascendante, avec la dist actuelle.
Vu que on est sur le sujet des css .document_* , je me rends compte que sur la dist,
les styles par défaut des sont appliqués aux liens englobant l’image des docs, ce qui n’est pas toujours très joli je trouve
Après quelques test en fait, cette modiifcation impacte sur le cas des documents .center, du coup ça oblige a modifier les css de la dist, ou a ce que figure ajoute ses propres surcharges ce qui je crois avoir compris n’était pas forcément la volonté de Tetue… <figure class='spip_document_#ID_DOCUMENT spip_documents[ spip_documents_(#ENV{align})][ (#ENV{align})][ (#ENV{class})] spip_lien_ok'[ style='float:(#ENV{align}|match{left|right});[width:(#GET{width}|max{120})]px;'][ (#TITRE|ou{#DESCRIPTIF}|ou{#CREDITS}|oui)role="group"]> [<a href="(#GET{url})"[ class="(#ENV{lien_class})"] title='#TYPE_DOCUMENT - [(#TAILLE|taille_en_octets|attribut_html)]'[ type="(#ENV{lien}|?{#ENV{lien_mime},#MIME_TYPE})"]>]<img src='#GET{fichier}' alt='' width='#GET{width}' height='#GET{height}'>[(#GET{url}|?{</a>})][ (#TITRE|ou{#DESCRIPTIF}|ou{#CREDITS}|oui) <figcaption[ style='width:(#GET{width}|min{350}|max{120})px;']>[ <strong class='#EDIT{titre} spip_doc_titre'>(#TITRE)</strong>][ <div class='#EDIT{descriptif} spip_doc_descriptif'>(#DESCRIPTIF|PtoBR)[(#NOTES|PtoBR)]</div> ][<small class="#EDIT{credits} spip_doc_credits">(#CREDITS)</small> ]</figcaption>] </figure> Le plus simple je crois est donc de modifier dist/css/medias.css : figure.center > figcaption { margin-right: auto; margin-left: auto; text-align: center; } ça « me paraitrait » être un bon compromis et compat ascendante, avec la dist actuelle. Vu que on est sur le sujet des css .document_* , je me rends compte que sur la dist, les styles par défaut des sont appliqués aux liens englobant l’image des docs, ce qui n’est pas toujours très joli je trouve .spip_documents > a, .spip_documents > a:hover { background-color: transparent; border: none; }
Après quelques test en fait, cette modiifcation impacte sur le cas des
documents
.center, du coup ça oblige a modifier les css de la dist, ou a ce que
figure ajoute ses propres surcharges
ce qui je crois avoir compris n’était pas forcément la volonté de Tetue…
Modifier la dist pour l'adapter à un plugin ? Quand je lis ça, je me dis qu'il y a un truc qui cloche
Après quelques test en fait, cette modiifcation impacte sur le cas des
documents
.center, du coup ça oblige a modifier les css de la dist, ou a ce que
figure ajoute ses propres surcharges
ce qui je crois avoir compris n’était pas forcément la volonté de Tetue…
Modifier la dist pour l'adapter à un plugin ? Quand je lis ça, je me dis qu'il y a un truc qui cloche