Skip to content
This repository has been archived by the owner on Jul 3, 2022. It is now read-only.

taille d'image minimum #15

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

taille d'image minimum #15

wants to merge 2 commits into from

Conversation

r043v
Copy link

@r043v r043v commented Sep 5, 2011

salut, j'ai modifié le plugin pour ne pas faire de la merde dans le cas de minuscules images, donc une option de plus qui sera la taille minimum en x et y, si à 0 c'est comme avant, sinon, c'est "zoomé" à cette taille, puis cropé suivant l'écran

j'ai aussi rajouté un padding entre l'image et le bord de la box, en option également, pour, pareil, faciliter la vie en cas de très petites images

également, j'ai détruit le cache d'image de la galerie à l'appel de la fonction principale, vu que je chargeais mes objets source en ajax, je l’appelais de multiple fois sur la page et la galerie conservais les vieilles images.

aussi optimisé et nettoyé un peu une partie du code.

@Grafikart
Copy link
Owner

Je check ça et je merge.

Pour les padding je n'ai pas bien compris, idéalement ne faudrait-il pas avoir minWidth et minHeight et auto calculé les padding dans ce cas ?

@r043v
Copy link
Author

r043v commented Sep 5, 2011

le padding étais surtout pour que la galerie ne cache pas trop les toute petite images, après j'ai fait ca vite fait cet aprèm j'ai pas tout changé et checké non plus en profondeur :- )

exemple la : http://jtree.tk/select?f5#img/icons/

@r043v
Copy link
Author

r043v commented Sep 5, 2011

d’ailleurs j'ai rajouté le padding à la fin, et ca interfère avec le crop de la taille max de l'image, faudrait faire une petite correction

// max resize
var maxx = windowW()-40;
var maxy = windowH()-40;

j'ai mis l’écran moins 20px partout, mais le padding n'est pas compté dedans, c'est vraiment limite la pour les grandes image qui veulent déborder

je change ca demain.

@r043v
Copy link
Author

r043v commented Sep 5, 2011

ne merge pas j'ai laissé un blème avec les grandes images, le crop n'est pas bon avec, je corrige ça des que possible.

@Grafikart
Copy link
Owner

Je ferais des tests de toute façon avant le merge.

Par contre est-ce vraiment util de faire un lightbox pour ouvrir une icone ? XD

@r043v
Copy link
Author

r043v commented Sep 5, 2011

la je fait un gestionnaire de fichiers, tout doit être générique, l'utilisateur lui n’aura surement pas d’icônes, mais sait' on jamais

mais même sans parler d’icônes, une image comme ca http://jtree.tk/img/eco.png passe très mal avec la galerie activée (sans compter que les bouton suivant et précédent passe sous la galerie)

aussi, mettre une taille minimum aidera pour l'affichage de la galerie, sur les images peu larges celle ci est inutilisable, peu être faudrait' il mettre en place un contrôle faisant scroller la galerie suivant la position de la sourie dans celle ci ?

@Grafikart
Copy link
Owner

La gallerie scroll suivant l'image sélectionné, mais jpense faire une version alternative avec la gallerie en ruban en bas (du coup beaucoup plus utilisable peu importe le format)

@r043v
Copy link
Author

r043v commented Sep 5, 2011

le ruban pourrais être vraiment bien c'est vrai :)

une bonne chose à faire serais de peu être aussi "casser" le ratio des images trop trop large dans la galerie, bon tu me dira que ce n'est pas tous les jours que l'on y verra des images de 3px de haut pour 700 de large :P

@r043v
Copy link
Author

r043v commented Sep 5, 2011

si ca t’intéresse je charge mes images comme ça :

var isrc = "[url]thumbs/"+imd5+".png";
var img = $("< img >",{src:isrc,title:fname}).attr("data-md5",imd5);
img.error(function()
{ $.post("[url]template/img/mkthumbs.php",{file:ipath,md5:imd5},function(data)
{ if(data == "ok") img.attr("src",isrc);
else alert("path : "+ipath+"\nmd5 : "+imd5+"\nthumbs : "+isrc+"\nreturn : "+data);
});
}).load(function()
{ img.appendTo(content);
if(++ifilesload >= ifilesnb)
{ ajaxloader.hide();
$("a.jtreeimgview").zoombox();
}
});

ça permet d'esquiver les timer et de gérer les erreurs 404 (ici en l’occurrence je crée les thumbs si ils ne sont pas présents)

@Grafikart
Copy link
Owner

Ok j'ai passé la gallery en bas c'est plus pratique. Pour la taille des image ce n'est pas un comportement généralisé je n'ai pas trop envie d'alourdir le code de tout le monde, jvais mettre ça sur une branche.

@r043v
Copy link
Author

r043v commented Sep 10, 2011

c'est bien mieux comme ça, bravo !

maintenant le code du padding n'est plus vraiment nécessaire (sauf pour les flèches peut être, à taille x minime elle sont inutilisables, peut être les positionner hors de l'image ?)

après alourdir le code, tout est question de point de vue, le patch fait 40 lignes, tu gagnerais certainement plus à optimiser par ci par la, autant niveau perf que niveau taille

@Grafikart
Copy link
Owner

C'est le truc des padding qui me gênait un peu, je vais plutôt utilisé l'image en background si taille < taille mini ça prendra moins de ligne.

@rteuteu55
Copy link

Salut, l'idée de "r043v" avoir une taille minimum et maximum de l'image (activable en option) serait vraiment utile.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants