Re: [Rails-spinoffs] question about Effect.Highlight and IE 5.5

Wed, 09 Nov 2005 02:08:33 -0800

See the list of supported browsers here:
(this is the same as for Prototype)

If you want to support IE 5.5, just stick your own copy in an 
additional .js file you load in, and maybe share the completed
example here! :)

You can also try getting the unittests to run on IE 5.5
(I haven't bothered with this).

Cheers,
Thomas

Am 07.11.2005 um 19:27 schrieb Gregory Hill:

I’ve recently begun testing some of the scriptaculous functionality in IE 5.5, since my company wants me to support as many browsers as possible.  When I do an Effect.Highlight on a <tr>, it doesn’t quite work right.  I’ve figured out a few things, but I’m not sure whether you want to change them or not.  Here’s what I know:

In Effect.Highlight.setup (line 532 in the rc4 version of effects.js), it does an element.style.backgroundColor to determine the ‘restorecolor’.  This doesn’t seem to work in IE 5.5, and if you switch it to Element.getStyle(this.element, ‘background-color’) (as is done a few lines earlier when determining the ‘endcolor’), it works correctly.  Should this be changed?

 

That, however, doesn’t fix the problem.  The problem is that IE 5.5 seems to ignore setting the style.backgroundColor on TR elements.  If you apply it instead to all of its child TD elements, it will work correctly.  So, in the Effect.Highlight.update and Effect.Highligh.finish functions you would need to loop through all childNodes if the element is a TR and apply the same color to each childNode, as such:

(...) 

There was another browser that had the same issue as IE 5.5, I can’t remember if it was Opera 8 or Safari 1.3 at the moment, though.  And I haven’t tested the fix in that other browser, either.

 Anyway, as that will probably make the effect slower for everyone just to fix fringe cases, I’m not sure it’s desirable to add it.  Maybe we could do a browser test as well.  I thought I’d pass it along to see what you guys think.  Everything else I tested works in IE 5.5 just fine (and Effect.Highlight works if it isn’t a TR).

 Oh, and nothing works in IE 5.0 at all, in case anyone is wondering.  _javascript_ errors galore.  Even prototype.js doesn’t work, because IE 5 doesn’t seem to recognize the (‘value’ in opt) used to determine a SELECT box’s value.

_______________________________________________
Rails-spinoffs mailing list
[email protected]
http://lists.rubyonrails.org/mailman/listinfo/rails-spinoffs

Reply via email to