developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 11 years ago

Closed 11 years ago

Last modified 10 years ago

#963 closed bug (fixed)

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

Reported by: Surfoo Owned by: Surfoo
Priority: normal Milestone: Jelix 1.1.4
Component: jelix:utils Version: trunk
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

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

Attachments (1)

imagemodifier.patch (1.6 KB) - added by Surfoo 11 years ago.

Download all attachments as: .zip

Change History (12)

Changed 11 years ago by Surfoo

comment:1 Changed 11 years ago by Surfoo

  • review set to review?
  • Status changed from new to assigned

comment:2 Changed 11 years ago by Lipki

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

comment:3 Changed 11 years ago by 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.

comment:4 Changed 11 years ago by Lipki

Je lance des recherches :)

comment:5 Changed 11 years ago by 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.

comment:6 Changed 11 years ago by 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.

comment:7 Changed 11 years ago by 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 ?

comment:8 Changed 11 years ago by Lipki

  • Documentation needed set
  • Resolution set to fixed
  • review changed from review? to review+
  • Status changed from assigned to closed
  • Version changed from 1.1.3 to trunk

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.

comment:9 Changed 11 years ago by Lipki

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:10 Changed 11 years ago by laurentj

  • Resolution set to fixed
  • Status changed from reopened to closed

comment:11 Changed 10 years ago by laurentj

  • Documentation needed unset
Note: See TracTickets for help on using tickets.