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

Gzyg

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

danielsan

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); ?>

Gzyg

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

bazooka07

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.

danielsan

pour le champ alt, perso j'utilise celui de l'article, vu que c'est associé ...

Gzyg

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

danielsan

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éphane

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

Gzyg

Sauf que je ne suis pas développeur et que je ne comprends pas d'où sort ce code.
Et je ne veux pas le savoir.
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éphane

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

danielsan

après c'est vrai que la fonction artThumbnail est d'afficher l'image ... et non une image dans un calque
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éphane

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

Gzyg

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

danielsan

mettre un div + une class + un style css alors qu'on peut directement viser "article section>img" ou un truc du genre ...

bazooka07

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;".

danielsan

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éphane

suppression de la div autour de l'image d'accroche pris en compte
https://github.com/pluxml/PluXml/commit/e1b559d61b32cdce589a34e7d9348ee88ccdc07f

Stéphane

ajout des champs title et alt sur l'image d'accroche
https://github.com/pluxml/PluXml/commit/dcf3f17e6de85b7baedf19a57df8c6a742b9b319

Gzyg

Testé, approuvé.

Merci.


à plus,

Gzyg

Stéphane

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: