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 [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/prototype-scriptaculous?hl=en.


Reply via email to