Impossible d'utiliser les attributs width et height sur le plugin image

[Surfoo]

Les attributs width et height ne sont pas listé comme utilisable dans le code source html, alors que ça peut être utile.

Et l'utilisation de l'homothétie via le paramètre de transformation "omo" s'affichait dans le code html par omo="1".

[Lipki]

Pour les width et height c'est volontaire, il doit y avoir une explication dans les nombreux autres tickets précédent.

[bballizlife]

Je ne trouve pas les explications qui ont amené à ne pas pouvoir modifier les attributs height et widh et en réfléchissant je n'en vois pas non plus.

Si quelqu'un a une explication valable je suis donc preneur. Sinon cela me semble tout à fait logique de pouvoir spécifier les dimensions de l'image.

Sinon ok pour omo cela se doit d'être corrigé.

En conclusion : si personne n'amène d'arguments contre height et width, ce patch est ok pour moi.

[Lipki]

Je lance des recherches :)

[Lipki]

Alors ...

Les attributs width et height sont utilisé dans le cas ou la redimension de l'image via php n'est pas possible. (gd indisponible)

La listes des attributs html utilisable a était créer a partir du ticket #587. Basé sur la liste des attributs valide/strict. Les attributs width et height étant utiliser comme attribut jtpl, il ne font pas partie de cette liste (sinon ça fait double emploi).

Maintenant le principe de ce plugin, c'est de faire du traitement sur l'image, pas de générer la balise html (c'est juste un plus). Je pense que si l'on veut afficher l'image en utilisant les attributs html width et height, il suffit de l'écrire a la main (c'est pas très long).

Pour ce qui est du patch je vais le tester, je pense qu'il va y avoir collision entre les attributs html, et les attributs jtpl.

[bballizlife]

Les attributs width et height étant utiliser comme attribut jtpl, il ne font pas partie de cette liste (sinon ça fait double emploi).

Je ne comprends pas ce que tu veux dire ici comme attribut jtpl.

Maintenant le principe de ce plugin, c'est de faire du traitement sur l'image, pas de générer la balise html

Justement si, c'est exactement un de ces objectifs, sinon ça ne serait pas un plugin jTpl qui génère justement cette balise img.

En fait je ne comprends pas ce problème de collision dont tu parle. Tu peux m'expliquer stp ? Parce que a priori je ne vois pas pourquoi on n'aurait pas les attributs width et height générés quoi qu'il arrive, si ils sont fournis. C'est un plus pour le navigateur pour faciliter son rendu il me semble.

[Lipki]

Je ne comprends pas ce que tu veux dire ici comme attribut jtpl.

attributs jtpl

{image 'truc.png', array('width'=>100, 'height'=>100, 'displaywidth'=>50, 'displayheight'=>50)}

attributs html

<img src="truc.png" width="100" height="100" />

---

En fait je ne comprends pas ce problème de collision dont tu parle. (...)

Je vois trois cas de figure :

• gd est absente, le script ne redimensionne pas l'image.
  La présence de width/height dans la balise html est indispensable.
  Déjà implémenté.
• L'image est redimensionné par le script.
  Je ne sait pas comment fonctionne les navigateurs, mais je suppose qu'il s'attende a ce que les attributs width/height redimensionne l'image, ce qui leur fait faire un traitement (ou une vérification) inutile.
• L'image est redimensionné par le script, mais on veut l'afficher d'une taille différente sur le site.
  La ok, les attributs width/height sont indispensable, mais il seront différent des attributs fourni dans la balise jtpl.
  C'est pour ça que je parle de collision.
  

ex :

{image 'truc.png', array('width'=>100, 'height'=>100, 'displaywidth'=>50, 'displayheight'=>50)}

<img src="truc.png" width="50" height="50" />

---

Maintenant est ce que Surfoo penser au cas numéro 2 ou au cas numéro 3 ?

Est-ce que c'est plus clair comme ça ?

[Lipki]

Alors j'ai les explications de Surfoo.

Il c'est servi des attributs width/height (html) en javascript.

Du coup (et comme on va pas y passer l'année) je suis d'accord pour valider le patch.

[laurentj]

pushed by baptiste:

• http://hg.jelix.org/jelix/jelix-trunk/rev/597484cb265d
• http://hg.jelix.org/jelix/releases/jelix-1.1.x/rev/0749d4dcde24