Safari .hide() on new elements fixed(?) in Safari - need opinion
Hi there,
I've been playing with creating new elements and fading them in on the
page (e.g. http://jqueryfordesigners.com/demo/image-load-demo.php) -
but there's a bug in jQuery & Safari that doesn't allow new elements,
outside of the DOM to have their display style set.
I know this is a low priority job in the bug tickets, but it's been
bothering me - so I've hacked around jQuery and *think* I've found the
fix.
Can I get someone to validate it in IE (I don't have a PC) or even
sanity check the change.
I've created a demo whereby the first link doesn't fade the image in
properly in Safari. The second link loaded in a patched version of
jQuery and the fade in effect works.
http://remysharp.com/demo/fadein.php (runs from 1.2.2)
http://remysharp.com/demo/fadein.php?patch=true (runs from patched
version of code)
I've tried to include other tests to make sure I've not broken
anything with the change - but it could do with some extra eyes.
The code change is as follows:
864,866c864
< for ( var a = elem; a && color(a); a = a.parentNode ) {
< if (jQuery.browser.safari && a.style.display === "")
< a.style.display = "none";
---