Annonce

#16 2010-06-10 21:31:00

thierry.f91
Membre
2010-04-20
669

Re: Un coup d'anti-bulles lol...

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


Le seul péché est de ne pas se risquer pour vivre son désir.

Hors ligne

#17 2011-05-12 17:18:54

flop25
Équipe Piwigo
2006-07-06
6544

Re: Un coup d'anti-bulles lol...

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

#18 2011-08-22 23:45:41

plg
Équipe Piwigo
Nantes, France, Europe
2002-04-05
12644

Re: Un coup d'anti-bulles lol...

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.


Les historiens ont établi que Pierrick était le premier utilisateur connu de Piwigo.

Hors ligne

#19 2011-08-23 17:13:16

flop25
Équipe Piwigo
2006-07-06
6544

Re: Un coup d'anti-bulles lol...

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 &quot;, ou même si pouvait poser problème

Hors ligne

#20 2011-08-23 17:52:14

plg
Équipe Piwigo
Nantes, France, Europe
2002-04-05
12644

Re: Un coup d'anti-bulles lol...

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.


Les historiens ont établi que Pierrick était le premier utilisateur connu de Piwigo.

Hors ligne

#21 2011-08-24 11:30:12

flop25
Équipe Piwigo
2006-07-06
6544

Re: Un coup d'anti-bulles lol...

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

#22 2011-08-24 11:31:29

plg
Équipe Piwigo
Nantes, France, Europe
2002-04-05
12644

Re: Un coup d'anti-bulles lol...

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)


Les historiens ont établi que Pierrick était le premier utilisateur connu de Piwigo.

Hors ligne

#23 2011-08-24 19:05:30

rvelices
Équipe Piwigo
2005-12-29
1417

Re: Un coup d'anti-bulles lol...

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.

Code:

{$thumbnail.TN_TITLE|@htmlspecialchars}

me semble plus judiciex

Hors ligne

#24 2011-08-24 19:20:45

plg
Équipe Piwigo
Nantes, France, Europe
2002-04-05
12644

Re: Un coup d'anti-bulles lol...

OK, je ferais l'équivalent, mais dans le PHP (pour la raison évoquée plus haut)


Les historiens ont établi que Pierrick était le premier utilisateur connu de Piwigo.

Hors ligne

#25 2011-08-24 19:22:54

plg
Équipe Piwigo
Nantes, France, Europe
2002-04-05
12644

Re: Un coup d'anti-bulles lol...

Quoique... si on a un name qui ressemble à :

Code:

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


Les historiens ont établi que Pierrick était le premier utilisateur connu de Piwigo.

Hors ligne

#26 2011-08-26 13:05:21

plg
Équipe Piwigo
Nantes, France, Europe
2002-04-05
12644

Re: Un coup d'anti-bulles lol...

Bon voilà, j'ai repris complètement le contenu de la fonction get_thumbnail_title. Voici un exemple de qu'on obtient maintenant.

http://fr.piwigo.org/forum/showimage.php?pid=175373&amp;filename=piwigo-2.3-tooltip.png


Les historiens ont établi que Pierrick était le premier utilisateur connu de Piwigo.

Hors ligne

#27 2011-08-26 13:06:31

plg
Équipe Piwigo
Nantes, France, Europe
2002-04-05
12644

Re: Un coup d'anti-bulles lol...

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.


Les historiens ont établi que Pierrick était le premier utilisateur connu de Piwigo.

Hors ligne

#28 2011-08-26 13:12:14

flop25
Équipe Piwigo
2006-07-06
6544

Re: Un coup d'anti-bulles lol...

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

#29 2011-08-26 14:09:50

plg
Équipe Piwigo
Nantes, France, Europe
2002-04-05
12644

Re: Un coup d'anti-bulles lol...

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 :

http://fr.piwigo.org/forum/showimage.php?pid=175377&amp;filename=piwigo-2.3-tooltip_long.png

Donc j'ai effectivement tronqué (à 100 caractères, dans mon exemple ça fait une quinzaine de mots).


Les historiens ont établi que Pierrick était le premier utilisateur connu de Piwigo.

Hors ligne

#30 2011-08-26 16:24:29

flop25
Équipe Piwigo
2006-07-06
6544

Re: Un coup d'anti-bulles lol...

ouhaaou impressionnant ^^

c'est parfait !

Hors ligne

Pied de page des forums

Propulsé par FluxBB

github twitter newsletter Faire un don Piwigo.org © 2002-2024 · Contact