When loading my site using IE8 I get the following JS error:
Webpage error details
User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 5.2; Trident/4.0; .NET CLR 1.1.4322; .NET CLR 2.0.50727; .NET CLR 3.0.4506.2152; .NET CLR 3.5.30729)
Message: Invalid property value.
Line: 184
Char: 5
Code: 0
URI: http://app.o-festivalen.com/dp6/sites/all/modules/lightbox2/js/lightbox....
I does not get any lightbox effects clicking on images.
In Firefox I don't get any error on page loading, and the lightbox effect works well on some images (but not with others - not sure if this problem is related to the JS-error - will do some more testing before I might file separate issue...)
By the way: Images i s handled using Image Assist together with Wysiwyg/TinyMCE (all latest dev versions).
Comment | File | Size | Author |
---|---|---|---|
#17 | color-fix.patch | 1.55 KB | jhl.verona |
#10 | patch-lightbox.js_.patch | 832 bytes | lnunesbr |
Comments
Comment #1
Duplika CreditAttribution: Duplika commentedI have the same problem but on line 181. It works flawlessly at Firefox and Chrome. Yet, on IE8 it doesn't.
Comment #2
Duplika CreditAttribution: Duplika commentedI have the same problem but on line 181. It works flawlessly at Firefox and Chrome. Yet, on IE8 it doesn't.
Comment #3
zarouk CreditAttribution: zarouk commentedHi all,
Same problem for me on IE8 on line 181.
Works fine on Firefox.
Best regards,
Comment #4
Countzero CreditAttribution: Countzero commentedSame problem with dev version and IE7, line 185 : 'invalid property value'.
This bug seems critical as it prevents at least some uses of the module with IE.
For example, the login functionality is totally broken : no popup and form displayed without styling in a normal window.
I also had to disable it on some pages because it prevented one custom script of mine to work.
Comment #5
diederik_2mpact CreditAttribution: diederik_2mpact commentedsubscribing
Comment #6
Javier CreditAttribution: Javier commentedSame problem here with dev version and IE7, lightbox is not working and I get the following error message:
Line:185
Character:5
'Invalid property value'
Hope someone can help.
Thanks in advance.
Comment #7
malphas CreditAttribution: malphas commentedI have the same problem for IE8 and IE6
Subscribing...
Comment #8
mixman CreditAttribution: mixman commentedI've tracked the problem down to the skin settings of lightbox2 - this is only an assumption, but I think that the default values for this section are invalid (empty) and jQuery is freaking out (undefined values). At least for IE7/8 of course (argh...)
Just go to /admin/settings/lightbox2 Advanced settings/ Skin settings and set the options:
- border size,
- lightbox color,
- font color
to some values (can't be empty).
Comment #9
nilsel CreditAttribution: nilsel commentedThanks Mixman! Had the same problems here, setting the skin border size and colors solved it right away, now the lightbox plays without js-errors in IE 6, 7 and 8 :) I'm using lightbox2-6.x-1.9.
Comment #10
lnunesbrAs mixman has said, the options - border size, lightbox color and font color must be set in the admin area, if not checked, returns a js error in IE8.
I actually made some update in the "lightbox.js" to prevent that, so, it checks if the respective variables really exists:
Lightbox.borderSize
Lightbox.boxColor
Lightbox.fontColor
And checking that, if undefined variables, prevents to make an invalid js call.
version: lightbox2-6.x-1.9
thx
leandro nunes
Comment #11
zilverdistel CreditAttribution: zilverdistel commentedsubscribing
Comment #12
YK85 CreditAttribution: YK85 commented+1 subscribing
is there a chance of this patch being committed? thanks!
Comment #13
chris_car CreditAttribution: chris_car commentedI had the same problem on IE7 and IE8 and setting skin values fixed the problem! Thanks!!!
Comment #14
joakimekroos CreditAttribution: joakimekroos commentedSkin values did the trick for me too. Thanks!
Comment #15
yousername CreditAttribution: yousername commentedSkin values it is!
Thank you!
Comment #16
jhl.verona CreditAttribution: jhl.verona commentedI have just spent a frustrating couple of hours figuring this out for myself - before reading the issue queue - sigh.
I can confirm that if the Lightbox color, Font color or Overlay color are missing, then a Javascript error occurs attempting to set backgroundColor = '#' for IE 8. Firefox ignores the error.
If you understand a little Italian (or just look at the code) I've explained my debugging process here: http://www.drupalitalia.org/node/10016#comment-33975
The patch in #10, while a good start, only cures the symptom, not the illness. These values should be validated in the module form code itself - but are not (CVS version DRUPAL-6--1-9).
HTH
John
Comment #17
jhl.verona CreditAttribution: jhl.verona commentedOk. Now I know why they weren't validated - an i18n slip and a typo. Patched against DRUPAL-6--1-9, which I think is version 6.x-1.9
(A small price to pay for such a useful module)
HTH
John
Comment #18
YK85 CreditAttribution: YK85 commentedComment #19
stella CreditAttribution: stella commentedpatch from #17 committed, thanks!
Comment #21
lovespell88 CreditAttribution: lovespell88 commentedI have found another annoying error in IE. Wonder if any of you guys encountered 'Expected identifier, string or number'. It will break all your JavaScript in IE. It's a nightmare to debug - even with Microsoft Script Debugger. And this javascript errror is only present in IE, you wouldn't have it in Safari, Opera, Chrome or Firefox