IE's version is redefined elsewhere.

Best,

Tobie

On Dec 1, 12:37 pm, Davide <daw...@gmail.com> wrote:
> HI All,
>
> yesterday while looking in the javascript source of JS I noticed that
> in the function getStyle: function(element, style) there is this piece
> of code.
>
> if (!value || value == 'auto') {
>    var css = document.defaultView.getComputedStyle(element, null);
>    value = css ? css[style] : null;
>
> }
>
> In my opinion there should be some probs with IE6 that does not have
> the function getComputedStyle().
>
> I would rather change the above with:
>
> if (!value || value == 'auto') {
>    var css;
>    if (window.getComputedStyle) {
>       css = document.defaultView.getComputedStyle(element, null);
>    } else if (element.currentStyle) {
>       css = element.currentStyle;
>    }
>    value = css ? css[style] : null;
>
> }
>
> What do you think ? Is this a resonable change ?

--

You received this message because you are subscribed to the Google Groups 
"Prototype & script.aculo.us" group.
To post to this group, send email to prototype-scriptacul...@googlegroups.com.
To unsubscribe from this group, send email to 
prototype-scriptaculous+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/prototype-scriptaculous?hl=en.


Reply via email to