Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Commit ae9136d

Browse files
committed
Moved the check for updates on start up to be done after the app is ready
1 parent f4101ce commit ae9136d

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

src/brackets.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -254,7 +254,6 @@ define(function (require, exports, module) {
254254
// check once a day, plus 2 minutes,
255255
// as the check will skip if the last check was not -24h ago
256256
window.setInterval(UpdateNotification.checkForUpdate, 86520000);
257-
UpdateNotification.checkForUpdate();
258257
}
259258
}
260259

src/utils/UpdateNotification.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -209,10 +209,6 @@ define(function (require, exports, module) {
209209

210210
updates.Strings = Strings;
211211
$updateList.html(Mustache.render(UpdateListTemplate, updates));
212-
213-
AppInit.appReady(function () {
214-
$dlg.find("button").focus();
215-
});
216212
}
217213

218214
/**
@@ -335,6 +331,11 @@ define(function (require, exports, module) {
335331
// Append locale to version info URL
336332
_versionInfoURL = brackets.config.update_info_url + brackets.getLocale() + ".json";
337333

334+
// Check for updates on App Ready
335+
AppInit.appReady(function () {
336+
checkForUpdate();
337+
});
338+
338339
// Define public API
339340
exports.checkForUpdate = checkForUpdate;
340341
});

0 commit comments

Comments
 (0)