search

Gregwar / Image

A PHP library to handle images
MIT License
1k stars 191 forks source link

Proposal: Adding image size difference in generation mobile platform, tablet or desktop #37

Closed jean-pasqualini closed 7 years ago

jean-pasqualini commented 11 years ago

It would be nice to go to resize a size chart to choose the size according to detected by the header sent by the client browser platform.

The functionality would be neutral detection device, is another component that is recommeded in the documentation for it.

Why I joined a switchable option parameter can be used on any action images.

Simply append an action Switch case sensitive and use the action as shown. eg

resizeSwitch( array( "phone" => array(150, 150), "tablet" => array(300, 300), "other" => array(450, 450), ), "phone");

jean-pasqualini commented 11 years ago

Bonjour, pouvez vous étudiez ma requête s'il vous plait ?

Gregwar commented 11 years ago

Bonjour,

Je dois avouer que je suis un peu sceptique, je ne sais pas vraiment si cette fonctionalité rentre dans les responsabilités de Gregwar\Image, on n'est plus trop dans la manipulation d'image mais plus dans le responsive design web

Et comme vous le montrez, il est possible de tout faire de l'extérieur, l'avantage n'est donc pas très clair

jean-pasqualini commented 11 years ago

La partie responsive n'est pas intégré dans le code du gregwar, seule la fonctionnalité de paramètre switchable compatible avec toutes les actions sur les images l'est justement car le responsive n'est pas de la responsabilité du gregwar.

Regarder donc le code du commit et vous comprendrez ce que je veux dire et l'intérêt que j'y trouve.

jean-pasqualini commented 11 years ago

Pour donner un example d'utilisation concret, vous proposer d’expliquer les fonctions suffixées plus en détails dans le readme ?

jean-pasqualini commented 11 years ago

Je vient de comprendre que tu parlait de la ligne 146, c'est sa ?

Si oui c'étais simplement pour expliqué une méthode compatible avec les ancienne version de gregwar mais requerant l'extension aop.

Si tu trouve que cette partie est inutile, je l'enleverais.

jean-pasqualini commented 11 years ago

J'attire votre attention sur notre discution

Gregwar commented 11 years ago

Bonjour, Cette pull request n'est plus mergeable automatiquement, il y a des conflits avec master Et il faudrait enlever la partie de la doc qui utilise php full power, je trouve que c'est en trop

jean-pasqualini commented 11 years ago

Bien, j’enlèverais donc la partie sur 'php-full-power' et je vais m'atteler à résoudre ce conflit en commencent par me mettre à jour vis a vis du dépôt officiel puis je reviendrais vers vous quand tous sera en règle.