Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
GoogleCodeExporter opened this issue Nov 17, 2015 · 1 comment

Comments

@GoogleCodeExporter
Copy link

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
Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant