Hi massa. I like your implementation - using a filter makes this contrib very flexible.

Am having a problem getting the images to display in IE6 & IE7. No problems with Firefox 2. Rotation works fine. Just nothing displayed where the image should be. Not even blank space. No doubt it will be a javascript incompatibility.

Thanks,
caign

Comments

jouke.postma’s picture

confirmed: IE6 gives as message:

line 7:
char 31:
Error: Object expected
Code: 0

brmassa’s picture

Assigned: Unassigned » brmassa
Status: Active » Needs review

caign and Jouke,

sorry for the waiting... i just posted a new update. it basicly corrects a javascript bug that firefox and konqueror (browsers that i use most) dont crash.

i ask you both to try and give me a feedback.

best regards,

massa

PS: its also available a translation file. if any of you know another language, it will be a pleasure to add a new translation.

presleyd’s picture

Still no images in IE6 or IE7.

kentford’s picture

I am also having trouble getting images to IE7. They appear to have height but no width, and I would rather not constrain. But more importantly, nothing works when I try to add a third or fourth image.
(on Foxfire)

brmassa’s picture

caign and Jouke, Christian and Kent,

its weird. i just tested on IE7 (drupal 4.7) and worked fine. my devel system is kubuntu linux, php5 mysql5 mainly.

on linux i used Firefox and Konqueror (Drupal 4.7 and 5)
on windows i tested on Firefox and Internet Explorer (Drupal 4 only)

i really need to know and solve this to you guys, but i will need more info. what server? PHP and drupal version? post here your slideshow code to let me test.

best regards,

massa

PS: i just posted a new version (1.13). But its mainly a code optimization and standardization. nothing related (i think) to this problem. just give it a test

kentford’s picture

Hi Massa, Thanks for helping figure this out. I have Drupal5.0beta and your version 1.13.

See error at:
http://wwsymposium.com/?q=node/22
Fine on Firefox, not on IE7 (on IE image has no width)

Below is code I am using. My earlier reported error with 3rd and 4th images was my error due to line return between images rather than . That part is fine now.

Thanks for helping figure this out!

Kent

[slideshow:2, rotate=5, blend=1, layout= reverse, name=, height= 200, width=, img=|http://wwsymposium.com/library/images/DiscussionGroups.jpg|LINK|Discussion Groups on Growing the Sport||, img=|http://wwsymposium.com/library/images/ascipark.jpg|LINK|Maryland's New Whitewater Park -Site of 2007 Symposium||, img=|http://wwsymposium.com/library/images/watertime.jpg|LINK|Inspiring On Water Instruction||, img=|http://wwsymposium.com/library/images/KeynoteTalksmallest.jpg|LINK|Thought Provoking Keynote Speaker||, img=|http://wwsymposium.com/library/images/RiverRescueInstructorTraining.jpg|LINK|Sharing best Instructional Techniques||, img=|http://wwsymposium.com/library/images/MikeMatheratwork.jpg|LINK|Rescue Training for Kayak Instruction||, img=|http://wwsymposium.com/library/images/InformalDiscussions.jpg|LINK|Informal Discussions||, img=|http://wwsymposium.com/library/images/classroom.jpg|LINK|ACA Instructor Updates||]

brmassa’s picture

kent,

1* on firefox and konqueror (on 4.7 and 5), your code works fine!
2* on my tests, even on IE7 the code worked.
3* on your site and on IE7, your code breaks: no width.
4* the fisrst cause: you put "width=," on your code. if you dont need to speficy the width, just omit it. you can omit almost every attribute, as rotate, name, height, name and blend. "width=," will be tranformed on width="" and IE consider it as 0.
5* why did it work on my system on IE7 but your site "crashed"? im really intrigued, since version 1.13, the module checks if width/height/name is determined to print it on the page. are you really using 1.13 for drupal 5?
6* why are you using drupal5beta if there is drupal 5.1 final? ehehe im curious.

bottom line: use 1.13 and the code
[slideshow:2, rotate=5, blend=1, layout= reverse, height= 200, img=|http://wwsymposium.com/library/images/DiscussionGroups.jpg|LINK|Discussion Groups on Growing the Sport||, img=|http://wwsymposium.com/library/images/ascipark.jpg|LINK|Maryland's New Whitewater Park -Site of 2007 Symposium||, img=|http://wwsymposium.com/library/images/watertime.jpg|LINK|Inspiring On Water Instruction||, img=|http://wwsymposium.com/library/images/KeynoteTalksmallest.jpg|LINK|Thought Provoking Keynote Speaker||, img=|http://wwsymposium.com/library/images/RiverRescueInstructorTraining.jpg|LINK|Sharing best Instructional Techniques||, img=|http://wwsymposium.com/library/images/MikeMatheratwork.jpg|LINK|Rescue Training for Kayak Instruction||, img=|http://wwsymposium.com/library/images/InformalDiscussions.jpg|LINK|Informal Discussions||, img=|http://wwsymposium.com/library/images/classroom.jpg|LINK|ACA Instructor Updates||]
and its gonna work! i guess...

regards,

massa

kentford’s picture

Hi massa,

Thanks for the help, which worked! My error in width was one of my attempts to try and get it to function, but clearly my string was the problem. Im on 5.0 beta because that is what I loaded originally, and I havent changed it yet.

Again thanks, I think it works great now.

Kent

brmassa’s picture

Status: Needs review » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)
dudelookslikealady’s picture

Status: Closed (fixed) » Active

Please check out http://iese2010.net . The slideshow on the front page appears right for opera and firefox but is cut into half for ie!! any reasons?

Please help.

dudelookslikealady’s picture

It appears correctly then shifts to the right ! :(

brmassa’s picture

Status: Active » Fixed

Guys,

its for 4.7 version right? im afraid to say that it not supported anymore. in fact, the only version i will support is for D6, since i changed the SsC API so deeply that i dont have the time to maintain two solutions.

regards,

massa

Anonymous’s picture

Status: Fixed » Closed (fixed)

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