Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Right Lightbox

#172

I'm sorry to say, but I think you misunderstood part of the Lightbox 'standard'...

When one wants to show an album using Lightbox, he shouldn't use lightbox[roadtrip], but lightbox[albumname], with albumname as anything he wants. This way he can use more than one album on one page.

roadtrip is just an example album name, not some magic word that automagically makes your lightbox links an album.

It could be that you did indeed understand the 'standard', and this is just your way of implementing it. In that case I suggest you set roadtripRule to something like a[rel='lightbox[]'], a[rel='lightbox[album]'] or a[rel='lightboxalbum]'] by default, as lightbox[roadtrip] makes no sense in most cases, only when the subject is, well, a roadtrip.

Reported by Douwe M. · January 20th, 2010 @ 05:10 PM

State: resolved
Milestone: none
Assigned to: MadRabbit MadRabbit

Activity

  1. MadRabbit
    MadRabbit

    hmm... honestly I didn't know that, will do something about that some day

    January 20th, 2010 @ 05:15 PM

  2. Douwe M.
    Douwe M.

    Hehe, ok :-) Shouldn't be too hard, as far as I can see...

    January 20th, 2010 @ 05:23 PM

  3. Douwe M.
    Douwe M.

    Another suggestion:

    Element.include({
  lightbox: function()
  {
    Lightbox.show(this);

    return this;
  }
});

    January 20th, 2010 @ 05:55 PM

  4. MadRabbit
    MadRabbit
    • State changed from new to resolved

    Implemented the multiple albums support feature. Will be published soon.

    March 19th, 2010 @ 07:26 PM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.