forum.coppermine-gallery.net

Dev Board => cpg1.4 Testing/Bugs => cpg1.4 Testing/Bugs: FIXED/CLOSED => Topic started by: Casper on April 21, 2005, 09:32:44 pm

Title: Filmstrip formatting broken
Post by: Casper on April 21, 2005, 09:32:44 pm

option1: (https://forum.coppermine-gallery.net/proxy.php?request=http%3A%2F%2Fwww.picture-parade.co.uk%2Fimages%2Ffs1.jpg&hash=800104916b1af1bdba5380ce2b1f6a199e892b6a)
option2: (https://forum.coppermine-gallery.net/proxy.php?request=http%3A%2F%2Fwww.picture-parade.co.uk%2Fimages%2Ffs2.jpg&hash=7279797b3ed130013907f974dd1254a476529cf7)

Title: Re: Filmstrip formatting broken
Post by: donnoman on April 22, 2005, 04:09:48 am

It's a difference between firefox and ie. In ie it looks fine.

I'll look into a solution.

Title: Re: Filmstrip formatting broken
Post by: donnoman on April 22, 2005, 04:17:17 am

Fix committed; check and see if it's acceptable.

Title: Re: Filmstrip formatting broken
Post by: Casper on April 22, 2005, 12:19:00 pm

The fix did make some difference, but still leaves very wide gaps between the thumbs when the thumbs do not fill the space, both FF and IE;

(https://forum.coppermine-gallery.net/proxy.php?request=http%3A%2F%2Fwww.picture-parade.co.uk%2Fimages%2Ffs1.jpg&hash=800104916b1af1bdba5380ce2b1f6a199e892b6a)

This is how it looked in earlier versions;

(https://forum.coppermine-gallery.net/proxy.php?request=http%3A%2F%2Fwww.picture-parade.co.uk%2Fimages%2Ffs2.jpg&hash=7279797b3ed130013907f974dd1254a476529cf7)

Another thing I discovered is when a theme has it's own tile.gif, that is not 30 px high, this is the result;

(https://forum.coppermine-gallery.net/proxy.php?request=http%3A%2F%2Fwww.picture-parade.co.uk%2Fimages%2Ffs3.jpg&hash=f9616cf649514b27876b43bdfd9bf0119204d6c0)

I have cured the width problem, by simply making this change in the filmstrip function;

Code: [Select]

starttable($CONFIG['picture_table_width']);
was changed to this;

Code: [Select]

starttable('');
To overcome the image height problem, I had to hardcode the filmstrip template into the theme.php, with the correct height.

Both the above changes result in what I beleive is the correct look for the filmstrip;

(https://forum.coppermine-gallery.net/proxy.php?request=http%3A%2F%2Fwww.picture-parade.co.uk%2Fimages%2Ffs4.jpg&hash=3d7a3074434770d9d28fcada3abcaab0d6c1df00)

The only change needed in the core code to achieve the width problem is the one to the function as mentioned above.  Shall I commit this to cvs?

Title: Re: Filmstrip formatting broken
Post by: donnoman on April 23, 2005, 02:17:06 am

This depends on what the look of the filmstrip is SUPPOSED to look like.

I personally don't like the lines of displayimage getting messed up when you have fewer than a full filmstrip.

I think it should remain = $CONFIG['picture_table_width']. Having the filmstrip magically changing demensions in a minority of cases (usually you'll have a full filmstrip) is a bad thing IMO.

In regards to the different height of a custom tile gif is introduced, I think thats what the template system is for and your fix of moving that item to the custom theme is appropriate. However, I'll take a look and see if there's a way we can easily accomodate and image of arbitrary size to be in the themes.inc.php.

Title: Re: Filmstrip formatting broken
Post by: Casper on April 23, 2005, 08:24:11 pm

This does not only affect people who have less thumbs set than will fill the width, but allalbums with less pics in them.  Also, where thumbs are of differing width, i.e., some portrait and some landscape.
I also repeat the way I suggest is the way it has displayed in previous versions.  This is a matter of taste, but I much prefer it the way I suggest, than having huge gaps between the thumbs, and the filmstrip edging overhanging the ends.

Title: Re: Filmstrip formatting broken
Post by: donnoman on April 24, 2005, 06:22:42 am

Unfortunately I can't agree, we need the other devs to chime in to get a consensus.

Title: Re: Filmstrip formatting broken
Post by: Tranz on April 24, 2005, 08:31:41 am

hmm... I think the filmstrip should be a consistent width, and instead of being spread out, the thumbnails should be left justified.

How are leftover images handled with real world negatives?

Title: Re: Filmstrip formatting broken
Post by: Joachim Müller on April 24, 2005, 09:37:43 am

both versions (the one donnoman is proposing and the "old" look Casper wants to have back) have an esthetical point (depending on the theme in use). As you pointed out, the behaviour can easily be changed in a custom theme, so the question is: "what should be the default behaviour of the core theme?" In the real world, a filmstrip negative has no big spaces, but is left aligned (so, the version Casper is suggesting is closer to the real world). As a workaround that should help all, this is my suggestion: make the default theme (core) display the filmstrip in the "traditional" way Casper is suggesting, and have at least one theme that has the filmstrip definition as suggested by donnoman (there should be a comment line in the theme file that says "change X to Y to make the fimstrip displayed in another way".

Joachim

Title: Re: Filmstrip formatting broken
Post by: donnoman on April 24, 2005, 08:40:07 pm

How about a compromise

Here is an example that combines my wishes and caspers.

Filmstrip matches the configured width, but the images are shoved together and centered in the filmstrip.

Title: Re: Filmstrip formatting broken
Post by: Casper on April 24, 2005, 08:48:02 pm

That would be better (to my eye) than the justified version.  So it seems compromise is the way  ;).

Title: Re: Filmstrip formatting broken
Post by: donnoman on April 24, 2005, 10:17:03 pm

I committed that change.

I also Added define('THEME_HAS_FILM_STRIP_GRAPHIC', 1); which will allow you to use just tile.gif instead of tile1.gif and tile2.gif.

I updated the theme documentation to reflect the new define.

The hieght for the tiles are no longer hardcoded, and the heights for tile1.gif and tile2.gif can be different.

Casper, you should be able to remove the filmstrip template from your theme.php and use the above define instead.

[edit]
I also stole your tile.gif for my powerbookg4 theme  ;D
[/edit]

Title: Re: Filmstrip formatting broken
Post by: Joachim Müller on April 24, 2005, 11:29:26 pm

[ot]
Anybody like the tile graphics I use on my personal site (http://gaugau.de/galerie/?lang=english)?:
(https://forum.coppermine-gallery.net/proxy.php?request=http%3A%2F%2Fgaugau.de%2Fgalerie%2Fthemes%2Fgaugau%2Fimages%2Fstrip_top.gif&hash=538a8a619ff30631feeac4a3353b1ff33d85e50b)

(https://forum.coppermine-gallery.net/proxy.php?request=http%3A%2F%2Fgaugau.de%2Fgalerie%2Fthemes%2Fgaugau%2Fimages%2Fstrip_bottom.gif&hash=f127f45e26a074051a3ce6958c52d52906d60352)

Feel free to add to a coppermine theme if you like (as proof of concept).

Joachim
[/ot]

Title: Re: Filmstrip formatting broken
Post by: Casper on April 25, 2005, 12:05:49 pm

I cannot claim credit for those filmstrips, they were posted, with others, a year or more ago, I cannot remember who by.

But so others can use them, here they are again, a selection of 30px and 20px filmstrip gifs.