[SPIP Zone] r102664 - in _plugins_/figure

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

Details: Connexion · GitLab

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 ?

--
RealET

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

Details: Connexion · GitLab

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 :wink:

Voir l’exemple (perdu parmi les autres) à cette page :

-- tetue

Le 06/02/2017 à 00:53, tetue@rezo.net a écrit :

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

Details: Connexion · GitLab

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 :wink:

Voir l’exemple (perdu parmi les autres) à cette page :
L’attribut title : à consommer avec modération - 24 jours de web

-- tetue

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

Bonjour Tetue,

Merci pour le plugin déjà :wink: ç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 ?

--
Bonne journée
Arnaud B. (Mist. GraphX)

Hi Arnaud,

Bonjour Tetue,

Merci pour le plugin déjà :wink: ç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 d’avoir testé :slight_smile:

-- tetue

Le 23/02/2017 à 16:23, tetue@rezo.net a écrit :

Hi Arnaud,

Bonjour Tetue,

Merci pour le plugin déjà :wink: ç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 d’avoir testé :slight_smile:

-- tetue

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

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 :wink:
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.

Merci pour le plugin :wink:

--
Bonne journée
Arnaud B. (Mist. GraphX)

Pour continuer sur les spip_doc_truc :

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.

--
RastaPopoulos

Le 14/03/2017 à 13:12, RastaPopoulos a écrit :

Pour continuer sur les spip_doc_truc :

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…

--
Bonne journée
Arnaud B. (Mist. GraphX)

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

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; }

Hop,

Le 14/03/2017 à 19:02, Mist. GraphX a écrit :

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 :slight_smile:

++
b_b

Le 14/03/2017 à 19:08, Bruno Bergot a écrit :

Hop,

Le 14/03/2017 à 19:02, Mist. GraphX a écrit :

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 :slight_smile:

++
b_b
----
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone

:wink: j'entends bien

dans l'idée c'est ça qui devrait être le standard des modeles medias, non ?

le truc c'est plutot de se pencher sur les css spip_document

Bonne journée
Arnaud B. (Mist. GraphX)

sinon y’a ça comme possibilité

<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 class="spip_doc_legende"[ style='width:(#GET{width}|min{350}|max{120})px;[(#ENV{align}|=={center}|oui)margin: 0 auto;]']>[
        <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>