Filmstrip formatting broken Filmstrip formatting broken
 

News:

cpg1.5.48 Security release - upgrade mandatory!
The Coppermine development team is releasing a security update for Coppermine in order to counter a recently discovered vulnerability. It is important that all users who run version cpg1.5.46 or older update to this latest version as soon as possible.
[more]

Main Menu

Filmstrip formatting broken

Started by Casper, April 21, 2005, 09:32:44 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Casper

option1: (https://coppermine-gallery.com/forum/proxy.php?request=http%3A%2F%2Fwww.picture-parade.co.uk%2Fimages%2Ffs1.jpg&hash=800104916b1af1bdba5380ce2b1f6a199e892b6a)
option2: (https://coppermine-gallery.com/forum/proxy.php?request=http%3A%2F%2Fwww.picture-parade.co.uk%2Fimages%2Ffs2.jpg&hash=7279797b3ed130013907f974dd1254a476529cf7)
It has been a long time now since I did my little bit here, and have done no coding or any other such stuff since. I'm back to being a noob here

donnoman

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

I'll look into a solution.

donnoman

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


Casper

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://coppermine-gallery.com/forum/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://coppermine-gallery.com/forum/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://coppermine-gallery.com/forum/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;

starttable($CONFIG['picture_table_width']);

was changed to this;

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://coppermine-gallery.com/forum/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?
It has been a long time now since I did my little bit here, and have done no coding or any other such stuff since. I'm back to being a noob here

donnoman

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.

Casper

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.
It has been a long time now since I did my little bit here, and have done no coding or any other such stuff since. I'm back to being a noob here

donnoman

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

Tranz

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?

Joachim Müller

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

donnoman

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.


Casper

That would be better (to my eye) than the justified version.  So it seems compromise is the way  ;).
It has been a long time now since I did my little bit here, and have done no coding or any other such stuff since. I'm back to being a noob here

donnoman

#11
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]

Joachim Müller

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

(https://coppermine-gallery.com/forum/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]

Casper

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.
It has been a long time now since I did my little bit here, and have done no coding or any other such stuff since. I'm back to being a noob here