Hello there,

The Gallery Formatter does not work at all with IE6 ( I will attach a screenshot later).
I do not use IE6 myself but the problem is that many of my users are from China. And in China, they mostly use hacks of the IE6 code: maxthon, 360 etc... I tried to tell them it can't be good to use a 14 year old code, written before mobile phones were hardly invented... but there is over 1 billion of them and I cannot shout so loud :):)
So, is there anyone that could help with a IE6 fix? It would be greatly appreciated.

Many thanks,
JJ

CommentFileSizeAuthor
#4 ie8.jpg197.82 KBDelta Bridges

Comments

Delta Bridges’s picture

Sorry to say, but just found out that the gallery also does not work with IE8 (:
I will check tomorrow with IE7 ( I do not have on my current computer).
No problem Chrome and Firefox :)
Cheers,
JJ

Delta Bridges’s picture

Title: IE6 and Chinese made browsers » Gallery formatter does not workl with IE6 and IE8 browsers
manuel garcia’s picture

Last time i checked on ie8 it was working, though i guess it couldnt hurt to check again. I dont have a windows box, only a VM runing ietester for this porpuse, so who knows.

I won't be supporting IE6 im affraid, but I will take patches if people work on it.

Delta Bridges’s picture

StatusFileSize
new197.82 KB

Dear Manuel,
I understand for IE6, no problem, I will try to find a way for this.
Here is a screenshot of the gallery with IE8 though. As you can see, the pictures are next to each other hence breaking also the tables of the website.
Please let me know if you need any further info to debug, I will be happy to provide what i can to help :)

manuel garcia’s picture

I've just tested IE8 runing on XP, and the gallery works fine.

Looking at your screenshot Delta Bridges, it looks like the JS did not run, are you seeing any JS errors?

manuel garcia’s picture

Title: Gallery formatter does not workl with IE6 and IE8 browsers » Gallery formatter does not work with IE6 and IE8 browsers
Version: 6.x-1.0-rc1 » 6.x-1.x-dev
Status: Active » Needs review

OK, I found why the JS was dieing on some versions of IE, and then discovered another phantom bug in all IEs, which I think I have fixed.

I have committed both changes (minor changes realy), please review the latest dev when it comes out, see if you can find any other problems. If you are anxious, just grab the current DEV release and apply these two patches: 1 2. I think dev releases are re-generatted every 24 hours by the drupal.org bot.

Delta Bridges’s picture

Thanks for the prompt actions Manuel. I will test the latest dev when it is out and let you know the outcome.
Thanks again :)
JJ

kirilius’s picture

I can confirm that the formatter does not look good in IE6. Many large companies still use IE6 because they have many house-grown Intranet applications that work only on IE6 so I don't see it going away soon.

In my case the carousel below the slide is very long: as long as the combined widths of all my thumbs! In fact it it not a carousel because I see all the thumbs, which spill over my screen and I need to scroll to the right. The same is true for the image description overlay.

manuel garcia’s picture

@kirilius: Like I said on comment #3, IE6 is not supported by this module. I welcome patches on it, but I won't be supporting a 14 year old browser sorry.

Delta Bridges’s picture

Title: Gallery formatter does not work with IE6 and IE8 browsers » Gallery formatter does not work with IE6 browsers

We understand Manuel, but do you mind to leave this issue open for now, in case a IE6 guru sees it and offers a fix?
Many thanks,
JJ

manuel garcia’s picture

Status: Needs review » Closed (won't fix)
johnpitcairn’s picture

It's very likely the problem is css-related. If someone would care to post an example URL that is broken in IE6 but displays as expected in IE7+, Firefox and Safari, I'll take a look.

manuel garcia’s picture

the long thumbnail section could be fixed by setting the wrapper inside the thumbs div to have an overflow of hidden, but stil I think the carousel doesnt work. Sorry johnpitcairn, but atm there is no demo... anyone has space on a server to get up a demo?

manuel garcia’s picture

Status: Closed (won't fix) » Postponed

I'm setting this to postoponed only so that people find out about this, and if someone wants to help, can do so here posting patches etc.

I have also setup a demo page with the latest dev, which I will be updating as commits get in: http://gf.ekhosting.net/

Delta Bridges’s picture

Thanks Manuel!
johnpitcairn, do you still need the screenshots? Let me know if you do.
I am willing to chip in 25 USD for a patch that would make this great module work with IE6
Many thanks:)
JJ

johnpitcairn’s picture

Yes, the screenshots would still be useful. Include notes describing exactly what is broken, even if you think it's obvious. I'll take a look in the next day or so, try to figure out whether a fix looks possible, and how long it might take.

johnpitcairn’s picture

Manuel: is it possible to grab a zipped package of the demo page (with db dump) so I can set it up locally?

manuel garcia’s picture

Yes it is possible, but it will take me more time to reset passwords, export the db and tarball it all than for you to setup a local test install.

AdrianB’s picture

It would be nice if the project page stated that IE6 is not supported (unless someone provides a patch, of course) so those who are forced to support IE6 can move on to similar modules.

johnpitcairn’s picture

Hi - sorry, a couple of projects have commenced at the same time. I'm unlikely to have time to look at the problem for some time.

Delta Bridges’s picture

No problem, thanks for letting us know... anybody else willing to help?
Thanks :)
JJ

bgogoi’s picture

Issue tags: +overflow

IE6 Problem at a dev site that I am trying to integrate this module: http://bit.ly/b3R04u

in FF and Google Chrome .. it works fine.

I tried overriding with

.gallery-thumbs {overflow: hidden;}

But it did shorten thumbnail row but the carousel wont work :(

It looks more than css issue.

** This would have given me a solution:: http://drupal.org/node/826928#comment-3115604

manuel garcia’s picture

Issue tags: -overflow

Yes, it is more than a plain CSS problem. The carrousel (JS) doesnt work in IE6.

halloffame’s picture

Any other similar modules that support ie6?

On the other hand would like to see this working on IE6. I'm willing to throw in some bounty to get this working on ie6. Any takers?

Anybody else wants to chip-in?

Delta Bridges’s picture

I am willing to chip in ... but if it is a carousel problem, it may be a little complicated.

skylord’s picture

Great module and it's a pity that author won't support all popular browsers. Many people still use IE6 and we should have this in mind as developers - we do our job not for ourselves but for users. :-)

With few patches 6.x-1.0-rc3 works OK in IE6 - as for me. Please, test it on your enviroments and give a feedback.

1) In template remove inline width styling for "gallery-thumbs" div. So it will look just like "<div class="gallery-thumbs">".
2) Add some CSS in default greenarrows style:

.galleryformatter-greenarrows .gallery-slides
{
  position: relative;
}

.galleryformatter-greenarrows .gallery-thumbs
{
  zoom: 1;
}

.galleryformatter-greenarrows .gallery-thumbs .wrapper
{
  width: 100%;
}

Carousel works OK, non-JS downgrade also works as expected.
The issue is with images - IE6 doesn't support transparent PNGs and "AlphaImageLoader" replaces doesn't support CSS sprites. So, with default images there are gray backgrounds on them. You need to make separate individual images for IE and apply appropriate styles... If anyone interested and solution above works OK for you - i will attach a full bundle.

2 Delta Bridges: Where are my 25 bucks? He-he... ;-)))))

knigh7’s picture

+1

manuel garcia’s picture

Status: Postponed » Closed (duplicate)

Let's close this one and work on it here: #1152266: IE6 Support, where someone submitted a patch. Testers are very welcome!