IE's version is redefined elsewhere. Best,
Tobie On Dec 1, 12:37 pm, Davide <[email protected]> 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 [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.
