doekman / base2

Automatically exported from code.google.com/p/base2
0 stars 0 forks source link

DOMContentLoaded does not fire in Safari 3.0.4 (and possibly other versions) #136

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
DOMContentLoaded is implemented in Webkit since 525.13. 
(according to 
https://prototype.lighthouseapp.com/projects/8886/tickets/64-safari-now-has-domc
ontentloaded-event-use-it)

Safari 3.0.4 runs Webkit 523.

http://code.google.com/p/base2/source/browse/version/1.0.1/src/base2-dom.js?r=30
9#823

In line 823, all 5xx versions of webkit are excluded from the workaround, so 
DOMContentLoaded isn't implemented for Safari 3.0.4.

To fix, change the regex to something like:
"@!Gecko20([^0]|0[3-9])|Webkit52[5-9]|Webkit5[3-9]|Webkit[6-9]|Opera[19]|MSIE.+m
ac": 

Original issue reported on code.google.com by ced...@veerwest.com on 23 Feb 2011 at 10:09

GoogleCodeExporter commented 9 years ago
Have you verified this or are you just guessing after glimpsing at the source 
code?

Original comment by dean.edw...@gmail.com on 24 Feb 2011 at 12:14