Wind buttons for film_strip in displayimage.php

[Tranz]

you want me to check for the " random bug" ( no image to display ) when at the end of the album?
found another little one too, if nbr of pics of the album <= items in the filmstrip, both right and left arrows have to be disabled

Do you see the random bug too or is it just me?

The second one should be straightforward checking for the $pic_count <= $max_items

[Rodinou]

Another thing

The mouse pointer doesn't appear on the arrows ...

Code Select Expand


style="pointer : hand"


resolves it. I know, it's not necessary, but I think if a user doesn't see his pointer change, he can think it's not clickable.

I like very much ...

So ...


MY request

If dhtml change is "too hard" ... or "too long", is it possible the filmstrip contains a "whole thumbnails" of a page ?! So ... in my case I have
20 pics per page ... so the filmstrip could contain 20 pics ?!

Thanx for this mod (for me it's more a mod : it's a great evolution of Coppermine)

[skypix]

Tranz, I got the bug at the end of the images too, couldn't get it to go back with the intermed. pic buttons either

Site looks cool, are you running 1.4? 

[Tranz]

Another thing

The mouse pointer doesn't appear on the arrows ...

Code Select Expand


style="pointer : hand"


resolves it. I know, it's not necessary, but I think if a user doesn't see his pointer change, he can think it's not clickable.

I agree that that would improve the user interface. Do you know where that goes?

[Tranz]

Site looks cool, are you running 1.4?

Thank you. My main site (in my sig) runs 1.3.2. When I post examples in discussing 1.4, that would be 1.4.1.

As a user, I can't wait to run v 1.4x on my main gallery.

[nol33t]

Not really related, but something I noticed.  Instead of wrapping a href tag around your forward and backward buttons, why not just do this:

Code Select Expand

<img src="images/prev.gif" border="0" onclick="BrowseFilmstripPrev();">  You don't get a pointer finger when you hover over it, but you can add title="Click for a previous image" to display a mouseover comment.

Well you noticed good, cause it's indeed nicer now...

ok, if you want to add it again...

theme.inc.php:

Code Select Expand


$prev_buttons = <<<EOT_PREV
<img align="left" style="cursor:pointer;cursor:hand" src="images/rewind.gif" border="0" onclick="BrowseFilmstripFastPrev()" title="$fr_title">
<img align="left" style="cursor:pointer;cursor:hand" src="images/prev.gif" border="0" onclick="BrowseFilmstripPrev()" title="$r_title">
EOT_PREV;


Code Select Expand


$next_buttons = <<<EOT_NEXT
<img align="right" style="cursor:pointer;cursor:hand" src="images/fastforward.gif" border="0" onclick="BrowseFilmstripFastNext()" title="$ff_title">
<img align="right" style="cursor:pointer;cursor:hand" src="images/next.gif" border="0" onclick="BrowseFilmstripNext()" title="$f_title">
EOT_NEXT;
}


-matt-

[skypix]

Guys, at the risk of GauGau raining fire and brimstone down on my head, any guess how long it will be before 1.4 is released?  I'm gathering from following the thread that this hack won't work in 1.3.2 without some custom tweaking, kind of off limits/too difficult for us noobs, yes?

Thanks Matt for the great work, it looks awesome...guess I'll have to be patient too...

....is it here yet?   

[Tranz]

Not really related, but something I noticed. Instead of wrapping a href tag around your forward and backward buttons, why not just do this:

Code Select Expand

<img src="images/prev.gif" border="0" onclick="BrowseFilmstripPrev();"> You don't get a pointer finger when you hover over it, but you can add title="Click for a previous image" to display a mouseover comment.

Well you noticed good, cause it's indeed nicer now...

ok, if you want to add it again...

Poor matt with all these suggestions. (https://coppermine-gallery.com/forum/proxy.php?request=http%3A%2F%2Ftu2.net%2Femoticons%2Ficon_console.gif&hash=ade76d31f04c7200b4f2441a241fc9e3f96fdc5f) I'm guilty of it too. But we (https://coppermine-gallery.com/forum/proxy.php?request=http%3A%2F%2Ftu2.net%2Femoticons%2Ficon_love.gif&hash=007ca33759d6b5029c2712b7b02073de2e8db474) the work you've done on this.

[nol33t]

found another little one too, if nbr of pics of the album <= items in the filmstrip, both right and left arrows have to be disabled

The second one should be straightforward checking for the $pic_count <= $max_items

themes.inc.php:
find

Code Select Expand


    //padding when at the beginning of the strip


and add before

Code Select Expand


    //if number of pics <= filmstrip size
    if ($pic_count<=$max_items)
    {
$r=-1;
$f=-1;
    }
    else


About the random bug, it won't stay random too much longer..

-matt-

just saw your post

Poor matt with all these suggestions.  I'm guilty of it too.  But we  the work you've done on this.

no biggie, if i don't like it i'll know where to look to remove it
and i'm pleased people are pleased by it 

[Tranz]

Thanks, that fixed one thing and brought up something else...

If the filmstrip is "crowded", there is a whitespace-type element that gets pushed down, which causes extra vertical space between the thumbnail bottom and tile graphic.

You can see it from two albums. One is small, so the thumbnails are spaced far apart. The other one is below the $max_items count but is crowded. You can see the element I'm referring to when you "select all" on the page. The crowded page shows the element taking up a complete line whereas the smaller album, the element starts above the baseline of the image.

[nol33t]

I see what you mean yea..but the size for your main tables are set to 600px..and not in %, so that's kinda normal no...?
--edit:( ok nothing to do with pix/% ..what i really meant is that the filmstrip has a sized set in config ( just like in 1.3.2 ) and too many thumbs in a too small table never mix well..)
if it's not  & you want to improve / fix that ( tried but don't manage to focus reading html tags tonight ),
you could try to play with $template_film_strip in theme.inc.php, specifically this part : ( it's the one i've modified )

Code Select Expand


$template_film_strip = <<<EOT

....

<!-- BEGIN thumb_cell -->


-matt-

[Tranz]

You were right; changing it to % helped.

Thanks to your tip on where to look, I found the culprit, so I can go back to using px.

I deleted &nbsp; after this:

Code Select Expand

<a href="{LINK_TGT}">{THUMB}</a>

[nol33t]

awesome , and while you were correcting it & posting i was editing my post and losing myself in theories

about the &nbsp, looked at the CVS, i didn't add it, so maybe it had a use..( like adding space between thumbs to make it look like a real filmstrip..? if it's the case it would be better to play with the cellspacing of the table right above)

[Tranz]

about the &nbsp, looked at the CVS, i didn't add it, so maybe it had a use..( like adding space between thumbs to make it look like a real filmstrip..? if it's the case it would be better to play with the cellspacing of the table right above)

I was wondering about its importance, too. However, I do not see any obvious drawbacks to removing it. I checked in FF, IE, and opera. The photos are still nicely spaced apart. I played with the cellspacing... it had a side-effect of adding unwanted space to the top and bottom.

I think maybe the next version of this should just use divs so we can have better control over the elements. However, this is great already. 

[nol33t]

About the random bug, it won't stay random too much longer..

TranzNDance, could you please try this (worked for me):

themes.inc.php
find

Code Select Expand


    //padding when at the end of the strip
    else if ($pos+$start_pos > $pic_count)

and replace it with

Code Select Expand


    //padding when at the end of the strip
    else if ($pos +$start_pos+1> $pic_count)


filmstrip.php
find ( last line..)

Code Select Expand


echo display_film_strip($album, $cat, $pos);


and add before

Code Select Expand


$max_items = $CONFIG['max_film_strip_items'];
if ( $pos+ceil($max_items/2) > $pic_count )
{
   $pos = $pic_count;
}


-matt-

[Tranz]

I haven't seen that "no images to display" box, although I don't know how to replicate that bug.

Here's something else I have found, though:
1. go here
2. click fast rewind.

The buttons will disappear although there are more photos. The filmstrip shows 10, and there are 12 in the album.

[nol33t]

ok I see that you are a real good tester, cause that one could have take time to spot! 

sorry about it, it's my way of checking when displaying the arrows or not, which was a little more too "relax" ...

themes.inc.php:

replace

Code Select Expand


    if ($pic_count<=$max_items)
    {
$r=-1;
$f=-1;
    }

with

Code Select Expand


    if ($pic_count<=$max_items)
    {
$r=-1;
$fr=-1;
$f=-1;
$ff=-1;
    }


and replace

Code Select Expand


//need to display the prev buttons?
if ( $r != -1 ){

with

Code Select Expand


//need to display the prev buttons?
if ( $r != -1 || $fr != -1 ){


and same

Code Select Expand


//need to display the next buttons?
if ( $f != -1 ){

with

Code Select Expand


//need to display the next buttons?
if ( $f != -1 || $ff != -1 ){


see you for the next bug or the next feature?

-matt-

[Tranz]

That fixed it, thanks. (https://coppermine-gallery.com/forum/proxy.php?request=http%3A%2F%2Ftu2.net%2Femoticons%2Ficon_thumbs_up.gif&hash=f845a31009dafd4285db67da2f3f39ffbaaab71c)

It doesn't seem like we can include this into the upcoming release because there isn't enough time.

I understand that there will be disappointment.

The good news is that we are close to releasing a beta version and thus closer to using 1.4 on our sites. If we keep adding stuff, we'll never be able to release the product. :-\\

sigh, as a user, the release can't come soon enough. However, being on the dev team, it's hard to stop wanting to add things and stick to a timeline. Such a dilemma.

Until next time, this would be a great mod, though. 

[nol33t]

@Thu: That was a pleasure working with you on that one

@All: the mod is pretty easy to apply:
- some lines to add in script.js
- theme.inc.php to modify
- a new file to put at the root of your coppermine install

So when the 1.4x will be released, i'll post the modified files on this thread

-matt-

[Tranz]

@Thu: That was a pleasure working with you on that one

Same here.

So when the 1.4x will be released, i'll post the modified files on this thread

You could create a new thread once it is finalized as a mod. We can then move it over to the mods/hacks board. (We're still in feature requests now )