Hello,

I have been using your gallery and I have a small layout problem.
At the moment the layout is described using..

<table><tr><td><image></td></tr></table>

This has real disadvantages when it comes to page scaling and content wrapping. Sure I can define my columns.. but either way I lose. If I want the gallery to fit on a smaller width page, then I can define '3' columns, but that same page scaled up will have immense padding between images.

If I define 10 images, the gallery overlaps across the page content when the width of the browser becomes less than the gallery. Can you see the problem this technique creates?

Instead I would like to use.. 

<div><image></div>

That way I could define width and background etc.. in the

style and the images would drop to the next line when the page was scaled..

I tried jumping in amongst your output code.. but it was beyond my understanding.
Thankyou for your assistance.

Comments

vm’s picture

vm commented

added code tags around HTML in above post so it can be read.

hurben’s picture

hurben commented

Thankyou!

vacilando’s picture

vacilando commented
Version: 5.x-2.9 » 5.x-2.0-dev
Priority: Normal » Critical

See the dev version - it already has table-less design now.

However, I cannot release it yet because I cannot find (and have little time to look!) a way to center (horizontally and vertically) images in their 'cells' in CSS.

See the test gallery at http://www.vacilando.eu/test

1) The 5th image is a vertical one - but it is not centered. I tried to use float: center but that did not work. Any ideas?
2) Similarly, see the 12th image (second from the left in the third row) - it is a bit less high than the rest. How to center it in its field?

vacilando’s picture

vacilando commented
Status: Active » Fixed

I've fixed the horizontal align now (1). In dev version now; soon I will roll out all-new v3.0.

The issue in (2) (vertical align) is less critical, but of course it would be nice to solve it. I tried using vertical-align:middle; all over the place, but it does not work.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

hurben’s picture

hurben commented

Sorry I've taken so long to respond.. I wasn't getting my drupal notifications.. I just looked at your test gallery, looks great to me! I would still like to spend some time cleaning up the lightbox interface.. making it nicer etc..

I notice that when you click 'next image' it goes blank then changes size to accomodate for the new image, but then displays the previous image very quickly before the next one loads.. I hope you know what I mean.

If you want to throw me the code, I can have a go at making it cleaner.. I have heaps of experience with <ul> and <li> mostly for building menus..

vacilando’s picture

vacilando commented
Title: Divs instead of table? » Vertical alignment in CSS?
Version: 5.x-2.0-dev » 5.x-3.0
Priority: Critical » Minor
Status: Closed (fixed) » Active

Hi, Hurben, good to see you back.

The issue you describe seems to be Lightbox-related, plus it may happen only in your browser, as I do not see that behaviour (or perhaps I am not understanding). Try Lightbox's forum.

I would love you to have a look at the code - please do. Most important to me seems to be the vertical alignment -- see higher. Any solution for that?

hurben’s picture

hurben commented

Where can I find the most recent version?
I will do my best to make the css as clean as possible and get it back to you.. :)

I don't know why others aren't very interested in your module, it's a really perfect solution to a particular situation.. Take care.