[résolu] [suggestion] à propos de artThumbnail

GzygGzyg Member
Bonjour,

La nouvelle fonction d'affichage artThumbnail est sympa mais :

1. elle est au sein d'une div que je trouve inutile et surtout encombrante pour l'intégrateur ou le designer qui lui préfèrerait un autre environnement ;
2. l'attribut "alt" renseigne le nom de l'image ce qui n'est pas son rôle : son rôle (entre autre) est de décrire l'image pour les navigateurs vocaux à condition que cette image ne soit pas que décorative.

Questions :

1. peut-on enlever la balise div et juste laisser img (ce qui est parfaitement sémantique) ?
2. peut-on mettre un champ manuel pour vraiment renseigner l'attribut alt si nécessaire ?



Merci, à plus,

Gzyg

Réponses

  • danielsandanielsan Member
    janvier 2016 modifié
    hello

    tu peux renseigner un format d'affichage normalement
    [== PHP ==]
    <?php
    $format = '
    <a href="#thumbnail-'.$plxShow->artId().'" data-rel="popup" data-position-to="window" data-transition="fade" title="Cliquer pour agrandir '.$title.'">
    	<img class="thumbnail art_thumbnail" src="#img_url" alt="'.$title.'" />
    </a>
    ';
    $plxShow->artThumbnail($format); ?>
    
  • Salut,

    ça règlerait le point 1 (bien que je préfèrerais que ce soit natif dans class.plx.show et avoir simplement $plxShow->artThumbnail(); dans le thème. :)

    ça ne règle pas le 2. ]:D


    à plus,

    Gzyg
  • bazooka07bazooka07 PluXml Lead Developer, Moderator
    janvier 2016 modifié
    La chaine de format ci-dessous serait suffisante :
    [== PHP ==]
    $format = '<img class=""#img_class" src="#img_url" alt="#img_alt" />'
    

    Pour le texte alternatif (alt), on peut utiliser cette formule
    [== PHP ==]
    $motif = '#(?:\.tb)?\.(?:jpg|jpeg|png|gif)#i';
    $alt = ucfirst(preg_replace($motif, '', basename($imgUrl)));
    $row = str_replace('#img_alt', $alt, $row);
    
    Reste plus qu'à donner un nom significatif au fichier image.
  • pour le champ alt, perso j'utilise celui de l'article, vu que c'est associé ...
  • danielsan : je ne comprends pas... ?

    bazooka : le problème d'utiliser le titre de l'image (même significatif) risque d'être le séparateur de mots (s'il y en a un). Comment un navigateur vocal va-t-il le lire ?


    à plus,

    Gzyg
  • danielsandanielsan Member
    janvier 2016 modifié
    Gzyg a écrit:
    danielsan : je ne comprends pas... ?
    le champ thumbnail est en lien avec l'article. Donc la valeur de alt est le titre de l'article, dans mon cas de figure.
    [== PHP ==]
    $title	= plxUtils::strCheck($plxShow->plxMotor->plxRecord_arts->f('title'));
    
  • StéphaneStéphane Member, Former PluXml Project Manager
    alt (comme alternative) est le texte à afficher si l'image n'est pas affichable, et contient généralement le texte de l'image. il peut être vide si on affiche une image purement décorative sinon c'est fortement conseillé de renseigner cet attribut.
    le format proposé est un format par défaut. il peut être personnalisé comme le propose danielsan

    Consultant PluXml

    Ancien responsable du projet (2010 à 2018)

  • Sauf que je ne suis pas développeur et que je ne comprends pas d'où sort ce code. :o
    Et je ne veux pas le savoir. :D
    Ce qui se passe dans le core n'est pas de mon domane de compétence. J'essaie juste de faire un thème à partir des fichiers dispos dans le thème par défaut.

    Pas grave je ferais autrement. ]:D

    Merci à tous de vous y être intéressés, à plus.

    Gzyg
  • StéphaneStéphane Member, Former PluXml Project Manager
    Salut Gzyg

    C'est pour ça que ton avis est important, ça permets de trouver un compromis pour répondre au maximum de besoins de tout le monde.
    Donc n'hésite pas à donner tes opinions. C'est pas toujours facile de faire plaisir à tout le monde, mais j'essaye autant que possible de rendre PluXml le plus proche de ce que vous demandez pour faciliter la réalisation de vos sites

    Consultant PluXml

    Ancien responsable du projet (2010 à 2018)

  • après c'est vrai que la fonction artThumbnail est d'afficher l'image ... et non une image dans un calque :D
    je rejoins donc Gzyg dans sa suggestion, même s'il est rare que j'utilise le format par défaut !

    un peu comme les fonctions tagList et catList qui ne sont pas homogènes ... ;)
    [== PHP ==]
    function tagList($format='<li><a class="#tag_size #tag_status" href="#tag_url" title="#tag_name">#tag_name</a></li>', $max='', $order='')
    function catList($extra='', $format='<li id="#cat_id" class="#cat_status"><a href="#cat_url" title="#cat_name">#cat_name</a></li>', $include='', $exclude='')
    

    pauvre Stéphane, t'étais content quand je faisais des pizz' dans la montagne ? ]:D
  • StéphaneStéphane Member, Former PluXml Project Manager
    Gzyg a écrit:
    1. elle est au sein d'une div que je trouve inutile et surtout encombrante pour l'intégrateur ou le designer qui lui préfèrerait un autre environnement ;

    En revanche ça m’intéresse de savoir quel "autre environnement" tu vises dans l'utilisation de artThumbnail par rapport à ce qui est proposé.

    Englober l'image dans une div est peut être superflue en effet car on peut agir sur son comportement css par rapport à son conteneur parent. Donc:
    - soit je laisse la div proposé par le format par défaut en sachant que l'image est englobée dans une div: <div class="art_thumbnail"> et qu'on a la possibilité de la faire sauter en changeant le format dans l'appel de la fonction dans le thème.
    - soit je la supprime du format par défaut et je la mets dans les fichiers du thème. ce qui laissera la possibilité de mettre ce qu'on veut à la place pour les concepteurs de thèmes

    Je retiens également l'argument de danielsan qui est de dire que la fonction artThumbnail a (devrait avoir) le rôle d'afficher l'image et rien d'autre (donc pas de div)
    On peut contre-argumenter en disant qu'on est dans la classe plxShow qui est la couche (dans un modele 3tiers MVC) qui sert au formatage de l'affichage pour s'affranchir coté thème d'avoir à coder en dur plein de choses. D'où l’intérêt de proposer par défaut la div.

    Donc sans faire un débat à rallonge, juste pour avoir votre ressenti vous préférez avec ou sans la div dans le format par défaut proposé dans l'appel de la fonction artThumbnail. Mon avis après réflexion serait de supprimer la div. Quel est le votre ?

    C'est le moment de faire vos revendications tant qu'on est sur la version de dev 5.5 ;)

    Consultant PluXml

    Ancien responsable du projet (2010 à 2018)

  • Stéphane : j'avais envisagé un item de liste (li), le thumbnail devant représenter une série de miniatures (une sorte de tête de liste, donc) qui n'apparaitrait qu'en mode chapo plus mobile (avec un tas de media queries compliquées).

    Pour ce qui est de la div, sa suppression me semblerait plus logique : une div n'a plus aucun rôle en terme sémantique. Elle sert juste à créer des conteneurs de conteneurs pour le positionnement, si besoin.


    à plus,

    Gzyg
  • mettre un div + une class + un style css alors qu'on peut directement viser "article section>img" ou un truc du genre ...
  • bazooka07bazooka07 PluXml Lead Developer, Moderator
    L'absence du div donnera plus de souplesse.
    Perso, je n'aime pas le grand blanc à droite de l'image.
    Sans le div, on peut facilement "habiller l'image avec le texte avec ceci dans theme.css :
    [== CSS ==]
    .art_thumbnail { float: left; margin-right: 1em; }
    
    Si quelqu'un a besoin d'une dimension précise pour l'image, il sera toujours possible de faire un "display: block;".
  • bazooka07 a écrit:
    L'absence du div donnera plus de souplesse.
    Perso, je n'aime pas le grand blanc à droite de l'image.
    Sans le div, on peut facilement "habiller l'image avec le texte avec ceci dans theme.css :
    [== CSS ==]
    .art_thumbnail { float: left; margin-right: 1em; }
    
    +1
  • StéphaneStéphane Member, Former PluXml Project Manager
    suppression de la div autour de l'image d'accroche pris en compte
    https://github.com/pluxml/PluXml/commit/e1b559d61b32cdce589a34e7d9348ee88ccdc07f

    Consultant PluXml

    Ancien responsable du projet (2010 à 2018)

  • StéphaneStéphane Member, Former PluXml Project Manager
    ajout des champs title et alt sur l'image d'accroche
    https://github.com/pluxml/PluXml/commit/dcf3f17e6de85b7baedf19a57df8c6a742b9b319

    Consultant PluXml

    Ancien responsable du projet (2010 à 2018)

  • Testé, approuvé.

    Merci. :)


    à plus,

    Gzyg
  • StéphaneStéphane Member, Former PluXml Project Manager
    Gzyg a écrit:
    Testé, approuvé.

    Merci. :)


    à plus,

    Gzyg

    Avec plaisir. tes arguments m'ont convaincu et le fil de discussion avec les différents avis de chacun aussi :cool:

    Consultant PluXml

    Ancien responsable du projet (2010 à 2018)

Connectez-vous ou Inscrivez-vous pour répondre.