https://bugzilla.wikimedia.org/show_bug.cgi?id=50038

       Web browser: ---
            Bug ID: 50038
           Summary: OutputPage: Don't output mw.loader.state "loading" and
                    mw.loader.state "ready" for the same module
           Product: MediaWiki
           Version: unspecified
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: Unprioritized
         Component: Skin and page rendering
          Assignee: wikibugs-l@lists.wikimedia.org
          Reporter: krinklem...@gmail.com
                CC: krinklem...@gmail.com
    Classification: Unclassified
   Mobile Platform: ---

It looks like we currently output both for some modules.

Current output on a wiki with user and site css/js enabled[1] but empty:

<script>if(window.mw){
mw.loader.state({"site":"loading","user":"loading","user.groups":"loading"});
}</script>
<script>if(window.mw){
mw.loader.load(["mediawiki.action.view.postEdit","mediawiki.user","mediawiki.hidpi","mediawiki.page.ready","mediawiki.searchSuggest","mediawiki.page.watch.ajax","ext.vector.collapsibleNav","ext.visualEditor.viewPageTarget","skins.vector.js"],null,true);
}</script>
<script>if(window.mw){
mw.loader.state({"site":"ready"});
}</script>
<script>if(window.mw){
mw.loader.state({"user":"ready"});
}</script>
<script>if(window.mw){
mw.loader.state({"user.groups":"ready"});
}</script>

The first one with "loading" is always output. The last 3 with "ready" are
output by OutputPage::makeResourceLoaderLink if $module->isKnownEmpty is true.

We should probably optimise this to just output "ready" in the first place.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
_______________________________________________
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to