Last modified: 2014-02-19 16:58:59 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T55990, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 53990 - Lazy-loading the mediawiki.notification module when mw.notify() is called results in minor, but noticeable delay
Lazy-loading the mediawiki.notification module when mw.notify() is called res...
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
JavaScript (Other open bugs)
1.22.0
All All
: Normal normal (vote)
: 1.23.0 release
Assigned To: Bartosz Dziewoński
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-09-10 12:37 UTC by Bartosz Dziewoński
Modified: 2014-02-19 16:58 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Bartosz Dziewoński 2013-09-10 12:37:19 UTC
Lazy-loading the mediawiki.notification module when mw.notify() is called results in minor, but noticeable delay.

This is particularly jarring with the AJAX watching functionality; the star stops spinning, nothing happens for a while, and then the notification appears.
Comment 1 Gerrit Notification Bot 2014-02-15 16:17:21 UTC
Change 113540 had a related patch set uploaded by Bartosz Dziewoński:
Preload mediawiki.notification when we know we're going to need it soon

https://gerrit.wikimedia.org/r/113540
Comment 2 Gerrit Notification Bot 2014-02-19 09:04:08 UTC
Change 113540 merged by jenkins-bot:
Preload mediawiki.notification when we know we're going to need it soon

https://gerrit.wikimedia.org/r/113540
Comment 3 Bartosz Dziewoński 2014-02-19 16:58:59 UTC
That should be good enough for me ;)

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links