Closed (duplicate)
Project:
Gallery formatter
Version:
6.x-1.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
29 May 2010 at 04:36 UTC
Updated:
25 May 2011 at 20:37 UTC
Jump to comment: Most recent file
Comments
Comment #1
Delta Bridges commentedSorry 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
Comment #2
Delta Bridges commentedComment #3
manuel garcia commentedLast 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.
Comment #4
Delta Bridges commentedDear 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 :)
Comment #5
manuel garcia commentedI'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?
Comment #6
manuel garcia commentedOK, 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.
Comment #7
Delta Bridges commentedThanks for the prompt actions Manuel. I will test the latest dev when it is out and let you know the outcome.
Thanks again :)
JJ
Comment #8
kirilius commentedI 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.
Comment #9
manuel garcia commented@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.
Comment #10
Delta Bridges commentedWe 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
Comment #11
manuel garcia commentedComment #12
johnpitcairn commentedIt'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.
Comment #13
manuel garcia commentedthe 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?
Comment #14
manuel garcia commentedI'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/
Comment #15
Delta Bridges commentedThanks 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
Comment #16
johnpitcairn commentedYes, 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.
Comment #17
johnpitcairn commentedManuel: is it possible to grab a zipped package of the demo page (with db dump) so I can set it up locally?
Comment #18
manuel garcia commentedYes 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.
Comment #19
AdrianB commentedIt 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.
Comment #20
johnpitcairn commentedHi - 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.
Comment #21
Delta Bridges commentedNo problem, thanks for letting us know... anybody else willing to help?
Thanks :)
JJ
Comment #22
bgogoi commentedIE6 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
Comment #23
manuel garcia commentedYes, it is more than a plain CSS problem. The carrousel (JS) doesnt work in IE6.
Comment #24
halloffame commentedAny 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?
Comment #25
Delta Bridges commentedI am willing to chip in ... but if it is a carousel problem, it may be a little complicated.
Comment #26
skylord commentedGreat 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:
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... ;-)))))
Comment #27
knigh7 commented+1
Comment #28
manuel garcia commentedLet's close this one and work on it here: #1152266: IE6 Support, where someone submitted a patch. Testers are very welcome!