tosca a écrit:
thierry.f91 a écrit:
Je les trouves bien les infos bulle du forum ainsi que les drapeau on peux pas faire pareil sur les site piwigo ?
Il me semble que la question a été posée il n'y a pas bien longtemps ...
As-tu fait une recherche dans le forum ?
Sur ce coup là j'ai pas étais un bonne élève lol je vais répondre non... mais je vais le faire...
Hors ligne
Je déterre le post pour proposer une solution :
garder la fonction get_thumbnail_title()
garder l'association 'TN_TITLE' => get_thumbnail_title($row), (category_default.inc.php l112)
remplacer <img class="thumbnail" src="{$thumbnail.TN_SRC}" alt="{$thumbnail.TN_ALT}" title="{$thumbnail.TN_TITLE}">
par <img class="thumbnail" src="{$thumbnail.TN_SRC}" alt="{$thumbnail.TN_ALT}" title="{if isset($thumbnail.NAME)}{$thumbnail.NAME|@replace:'"':' '}{else}{$thumbnail.TN_TITLE}{/if}">
alors ?
Hors ligne
Je suis en train de bosser sur l'améliorer des attributs alt et title des miniatures. Et donc forcément, je tombe sur tes récents changements.
Je ne comprends pas bien à quoi sert $thumbnail.TN_TITLE vu le code dans thumbnails.tpl, puisque $thumbnail.NAME sera toujours rempli.
Je trouve ça bien qu'on ait les infos du nombre de visite ou du rating score dans l'infobulle selon la section où on se trouve, même si un "(1234 visites) Champignons dans les bois" seraient plus explicite que "(1234) Champignons dans les bois".
Et je ne comprends pas bien le principe du {$thumbnail.NAME|@replace:'"':' '} : que cherches tu à protéger ?
Personnellement, je préfèrerais qu'on remette l'ancien code pour le template et qu'on ne change que la valeur attribué à TN_TITLE dans le code PHP : inutile d'imposer des changements à tous les thèmes qui dérivent thumbnails.tpl. L'infobulle ne me semble pas être un élément distinctif d'un thème par rapport à un autre.
Hors ligne
plg a écrit:
Je ne comprends pas bien à quoi sert $thumbnail.TN_TITLE vu le code dans thumbnails.tpl, puisque $thumbnail.NAME sera toujours rempli.
justement pour
plg a écrit:
inutile d'imposer des changements à tous les thèmes qui dérivent thumbnails.tpl. L'infobulle ne me semble pas être un élément distinctif d'un thème par rapport à un autre.
Ma vision était soit de supprimer ce TN_TITLE soit de le changer/améliorer : utiliser le nombre de visites ou la notation c'est pas mal.
plg a écrit:
Et je ne comprends pas bien le principe du {$thumbnail.NAME|@replace:'"':' '} : que cherches tu à protéger ?
juste une mauvaise interprétation du à la présence de guillemets : mais je l'ai fait par réflexe sans regarder si le code remplaçait par ", ou même si pouvait poser problème
Hors ligne
OK, donc je propose qu'on remette l'ancien code au niveau du template mais qu'on change la valeur du TN_TITLE tout simplement.
Concernant la "protection" du TN_TITLE, il faut faire comme pour les meta du header.tpl, à savoir :
{$thumbnail.TN_TITLE|@strip_tags:false|@replace:'"':' '}
sauf que cela impose que tous les thèmes qui dérivent thumbnails.tpl doivent s'adapter, ce qui s'oppose à la tentative de minimiser l'impact de la 2.3 sur les thèmes. Donc il vaudrait mieux que ces protections soient faites en PHP.
Hors ligne
comme la 2.3 sort bientôt, je fais un revert et tu verras comment tu souhaites faire, si tu as le temps d'ici une semaine. Sinon 2.4
Hors ligne
OK, je fais ça avant la fin de la semaine (là je suis sur la refonte du processus de réinitialisation du mot de passe)
Hors ligne
plg a écrit:
OK, donc je propose qu'on remette l'ancien code au niveau du template mais qu'on change la valeur du TN_TITLE tout simplement.
Concernant la "protection" du TN_TITLE, il faut faire comme pour les meta du header.tpl, à savoir :Code:
{$thumbnail.TN_TITLE|@strip_tags:false|@replace:'"':' '}sauf que cela impose que tous les thèmes qui dérivent thumbnails.tpl doivent s'adapter, ce qui s'oppose à la tentative de minimiser l'impact de la 2.3 sur les thèmes. Donc il vaudrait mieux que ces protections soient faites en PHP.
{$thumbnail.TN_TITLE|@htmlspecialchars}
me semble plus judiciex
Hors ligne
Quoique... si on a un name qui ressemble à :
<span style="color:red">Mes champignons préférés</span>
On a l'air malin avec htmlspecialchars dans une infobulle, alors qu'avec le striptags, au moins ça nettoie de ce genre de cas. Qu'en penses tu ? As tu un exemple ou htmlspecialchars est plus utile ?
Hors ligne
Bon voilà, j'ai repris complètement le contenu de la fonction get_thumbnail_title. Voici un exemple de qu'on obtient maintenant.
Hors ligne
Au début, j'ai essayé de mettre des retours à la ligne dans le tooltip (et le résultat est sympa sous Chrome) mais ça ne fonctionne pas avec Firefox.
Hors ligne
sympa !
Je n'ai regardé le code mais juste une question à froid : pour les descriptions longues (là déjà je trouve limite), un |truncate:40:" [...]" serait pas mal non ? 40 étant à adapter
merci ;)
Hors ligne
Il me semblait que c'est géré par les navigateurs. Mais après un test d'une description très longue voici ce que ça donne :
Donc j'ai effectivement tronqué (à 100 caractères, dans mon exemple ça fait une quinzaine de mots).
Hors ligne
ouhaaou impressionnant ^^
c'est parfait !
Hors ligne