fx.accordion doesn't work on IE 8 and IE 7

[Suzanne]

fx.accordion doesn't work on IE 8 and IE 7. You will find the error
when opening
the Mootools demo on IE 8:

http://jsfiddle.net/api/post/mootools/1.4/dependencies/more/

[Arian Stolwijk]

Try mootools 1.4.3

[Suzanne]

I just upgraded to 1.4.4 to solve IE 7 crash (same issue in #2163). I
cannot go back since IE 7 crash is more severe. Is there any way I can
avoid the accordion problem without give it up (my app has been
deployed for two years)?

[Arian Stolwijk]

what about 1.4.1? You can also try to use the almost released 1.4.5, either from https://github.com/mootools/mootools-core/wiki/Build-MooTools-from-git-master or http://mootools.net/nightly/build/mootools-core.js

See this issue: https://github.com/mootools/mootools-core/issues/2160

Sorry for the inconvenience.

[Suzanne]

didn't find 1.4.1 download. I tried the night build. It solved IE 8
issue, but the same errors remain on IE 7.
Thanks for the help.

On Feb 20, 1:20 pm, Arian Stolwijk <stolwijk.ar...@gmail.com> wrote:
> what about 1.4.1? You can also try to use the almost released 1.4.5, either

> fromhttps://github.com/mootools/mootools-core/wiki/Build-MooTools-from-gi...
>  orhttp://mootools.net/nightly/build/mootools-core.js

[Suzanne]

Found and tested 1.4.1. my page shows more errors in IE 7 now.

[Olmo Maldonado]

ahh confusing. :D

can you provide us a jsfiddle with IE7 errors? we'll see if it's something new so we can create a github issue

[Suzanne]

I am new to jsfiddle and I don't know how to create script using
1.4.6dev. Can you try the demo
http://mootools.net/demos/?demo=Accordion with 1.4.6dev on your IE 7?

[Suzanne]

Hi Olmo,
Can you reproduce the error?

On Feb 20, 2:07 pm, Olmo Maldonado <olmo.maldon...@gmail.com> wrote:

[Gafa]

Suzanne,

In the 1.4.4 core file uncompressed find line 3771
"this.style[property] = value;" replace with

if !isNaN(value){
this.style[property] = value;
}

Thisis a band-aide fix, as the core issue still remains but this will
fix it for IE7 in the meantime.

Gafa

[Gafa]

Or

if (value !='NaN' || value!='Infinity'){
this.style[property] = value;

[Philip Thompson]

On Fri, Feb 24, 2012 at 6:56 AM, Gafa <sfred...@ireland.com> wrote:

if (value !='NaN' || value!='Infinity'){

You probably mean...

if (value != 'NaN' && value != 'Infinity') {...}

~Philip 

[Olmo Maldonado]

Why is the value NaN or Infinity?

[Suzanne]

I added if (value != 'NaN' && value != 'Infinity') {} and it cleared
out the error. Thank you!



On Feb 24, 9:42 am, Philip Thompson <philthath...@gmail.com> wrote:

[Olmo Maldonado]

Yes I understand, but I'm trying to understand when the Accordion code would have a NaN or Infinity value.

[Gafa]

Apparently the accordion code calling element.Implement... setStyle,
the "value" parameter in version 1.4.4 comes in as "NaN" IE7, and
adversely is sometimes set to "Infinity" in the (string(number))
calculation. Issue is with the value parameter, I suspect the code
prior. I didn't go beyond testing that.

Gafa

[Gafa]

good to know that helped.

[Olmo Maldonado]

Thanks, for reporting. We'll take a look and see where the fault starts.