I've run into an issue where the fade / blend doesn't work with the 'dir=' attribute unless 'img=|(image path)|(somelink)||||,' is set.

if: 'img=||||||,' is set, then the 'dir=|', the fade doesn't work
if: 'img=|(image path)|||||,' is set, then the 'dir=|'', the fade doesn't work

BUT

if: 'img=|(image path)|(some_url_path)|||||,' is set, the fade works...

It's no problem to set an image up with a blank url or something, but this might cause folks endless time in trying to figure out why the fade isn't working if they only set the 'dir=|',

I also tested: 'img=|(image path)|||||,' WITHOUT 'dir=', and the fade did not work...

module version: 1.24
Drupal version: 5.2

Hope that's enough info!

Comments

brmassa’s picture

philsward,

hmmm lets go:
1* why are you inserting "img=||||||," ? if there is no image, simply delete this tag. the "img" tag is not needed.

2* its a bit confusing examples. i ask you to post the real tests, ok?

regards,

massa

philsward’s picture

Well, I did as you said, took out the img="" tag, and guess what... no fade... lol

When I first posted this, I was messing around with it in a test environment, so my mind is a bit rusty on the workings of why I posted this in the first place. I think that what I was trying to get at, is the fact that the img='' tag HAS TO BE IN PLACE in order for the fade to work. On top of that, if you just set the img= or 'img=(image path)|||||', the fade won't work. The only way I was able to get the fade to work using the dir= tag, was to set the img= tag with a static image AND url path, as noted through the 'img=|(image path)|(some_url_path)|||||,'.

Here is my working fading code:

[slideshow: 2, rotate=4, blend=3, height=250, img=|http://xenitive.com/t/sandbox/files/images/nabsinc/Johnson_City_FCU.jpg|http://xenitive.com/t/sandbox/files/images/nabsinc/Johnson_City_FCU.jpg|||self|, dir=|files/images/nabsinc/|yes||Temporary and Permanent Banks|||]

Here is my non-fading code:

[slideshow: 2, rotate=4, blend=3, height=250, dir=|files/images/nabsinc/|yes||Temporary and Permanent Banks|||]

Long story short, when using the dir= tag, img= tag MUST be in place with the image path and url path set.

alynner’s picture

I found that it worked without the img= tag, but I had to have something in DIR_LINK.

I didn't want it to link to anything, so I commented out the two lines for $output_link and $output_link2 (lines 225-226). That made it not fade again. I think your code relies on the style or ID of that link. Perhaps you should just scrap "and $ss[$i]['link'] != 'javascript: void(0);'" from the if statement on line 224?

I don't like that its still a link though, so I just deleted the href from the link (on line 225) because I know I don't want them to be links anywhere in my site and it works all great.

cheers
alynner

brmassa’s picture

Status: Active » Fixed

Guys,

its fixed. It will be on the next version (only for D6)

regards,

massa

hanskuiters’s picture

Too bad it can't be fixed for D5 , since it's an old bug.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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

meeotch’s picture

Status: Closed (fixed) » Active

Sorry to re-open this... just wondering if there's any chance you could point me at the broken code, so I can try to fix this for Drupal 5? I think it's a great module (thanks!), but there's unfortunately no chance that I can update my entire site to Drupal 6 just to continue using it. (As I mentioned, I'll try to make the fix myself... just hoping you can point me in the right direction.)

brmassa’s picture

Status: Active » Fixed

meeotch,

im afraid to say that i wont support D5 versions anymore. I changed the API deeply on D6 and i simply dont have the time to maintain 2 different versions.

im really sorry.

regards,

massa

Anonymous’s picture

Status: Fixed » Closed (fixed)

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

ijustlost’s picture

Sorry to re-open this yet again. I've just spent over an hour trying to get fade to work before I saw this post. Could you at least note this in the known issues section of the readme. You could just link to this page - I'm sure this would save a lot of peeps a lot of time!

Cheers,