sebsauvage / MinigalNano

GNU Affero General Public License v3.0
161 stars 50 forks source link

Switch LightBox lib #44

Open qwertygc opened 10 years ago

qwertygc commented 10 years ago

http://tholman.com/intense-images/ vous en pensez quoi ?

tmos commented 10 years ago

Plusieurs choses :

Mais justement, elle est peut être un peu trop originale, dans le sens pas assez "passe partout". Je me vois mal envoyer une galerie de photos de mon week end pèche à mes grands parents avec ça quoi :/ Je ne sais pas si tous les utilisateurs se retrouveront avec cette lightbox.

Je pensais à celle-ci : http://brutaldesign.github.io/swipebox/ mais elle n'a pas de commentaires. J'avoue que ça commente à me saouler, il nous faut une lightbox :

À moins que quelqu'un ne trouve la perle rare, je sens qu'il va falloir se coder ça à la mano… J'ai vraiment pas mal cherché et je n'ai jamais rien trouvé qui allie de mon coté.

qwertygc commented 10 years ago

Pour l'exif, pourquoi pas faire du côté de PHP ? Et pour la galerie http://lychee.electerious.com/ (inspiration) http://nanogallery.brisbois.fr/#nanogallery/nanoGalleryHome/5852571211141627249/5852571230518020850

tmos commented 10 years ago

Lychee est une de mes galeries favorites, je la trouve vraiment magnifique :) +1 pour l'inspi donc ! Si j'ai bien saisi, lychee utilise nanogallery ? Ça peut être une très bonne option en effet.

On avait pensé prendre l'exif en charge côté php, mais juste à moitié : détecter l'orientation et ajouter une classe particulière sur l'image, la galerie serait en charge de tourner l'image ensuite. Ce qui demande une petite modif côté galerie. Pas trouvé mieux :(

qwertygc commented 10 years ago

Je sais pas si l'un utilise l'autre, c'est juste des idées balancées.

tmos commented 10 years ago

on dirait bien, en tout cas ils se ressemblent vachement je trouve. Bon va falloir que je me penche sérieusement là dessus, ça va entrainer un bordel monstre dans le code de tout reprendre ! Nano gallery à l'air très sympa comme lightbox, je vais y jeter un œil d'un point de vue technique :)

nodiscc commented 9 years ago

Petit rappel de la discussion sur https://github.com/sebsauvage/MinigalNano/pull/50:

We can and should remove mediabox. Then we just need a new lightbox to display images/video/audio elements I totally agree with @nodiscc about the conclusion : we should use a lightbox which leverage HTML5 Media API. We discussed about some lightboxes here : #44 . I thinrk that a lightbox using Jquery could be an easy and efficient solution (few lightboxes are in vanilla), as we can replace jquery with zepto, wich is really light.

tmos commented 9 years ago

I think this is THE perfect lightbox : http://photoswipe.com/


    image zoom for desktop and mobile (multi-touch)
    advanced gestures, like pinch or vertical swipe to close
    script is modular, you can exclude parts that you don't need
    default UI is optimised for desktop, for mobile and for devices with mixed input types (like Chromebook Pixel or Surface)
    pure Vanilla JS (no depencies, no jQuery required!)
    IE 8 support (just basic functionality)
nodiscc commented 9 years ago

It looks good. No HTML5 media (audio/video) support though.

tmos commented 9 years ago

This is true, but I think the main point of minigal is pictures. audio/video is bonus. And we still let aviable viode/audio for download, and maybe later, add an implementation on the lightbox itself. Why not ?

nodiscc commented 9 years ago

Right, it's better than the current situation anyway, adding video support in the future stays a possibility.

xannonmedia commented 8 years ago

Might I be so bold as to seriously suggest Venobox. Its awesome...

https://github.com/nicolafranchini/VenoBox