Bugfix: run AppInit._dispatchReady AFTER loading the extensions #1854

Merged
merged 1 commit into from Oct 16, 2012

3 participants

@DennisKehrig

This restores the intended behavior of what once was brackets.ready.
It could therefore be considered a fix for #1526, but please note the comments there.

@jasonsanjose jasonsanjose was assigned Oct 16, 2012
@jasonsanjose
Adobe Systems Incorporated member

Yikes. Thanks for catching that. Merging.

@jasonsanjose jasonsanjose merged commit b8fd540 into adobe:master Oct 16, 2012
@peterflynn
Adobe Systems Incorporated member

To clarify the situation with #1526 (per Dennis's notes there): APP_READY is still not guaranteed to fire after the initial document is loaded. It was a race condition before, and remains a race condition after this patch -- it's now somewhat more likely that APP_READY happens second, but depending on timing sometimes it could still come first.

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