mhershberger wrote:

Unassigning default assignments. http://article.gmane.org/gmane.science.linguistics.wikipedia.technical/54734

yes, WikiEditor can't load on any Gadget, only load on [[user:YourUserName/common.js]].

Plese fix it!

(In reply to comment #2)

yes, WikiEditor can't load on any Gadget, only load on
[[user:YourUserName/common.js]].

Plese fix it!

Restoring original values. Seems you misunderstood the original bug report.

We now show the CodeEditor on JS pages, which relies on the WikiEditor. This makes this bug less noticeable in WMF deploys, but it still exists.

Change 141296 had a related patch set uploaded by TheDJ:
Only add WikiEditor for content model WikiText

https://gerrit.wikimedia.org/r/141296

Change 141296 merged by jenkins-bot:
Only add WikiEditor for content model WikiText

https://gerrit.wikimedia.org/r/141296

Does the above merge mean the patch is now applied on https://test2.wikipedia.org ?

If so, then I'm guessing the fact the Page: namespace having a content-model of 'proofread-page' screwed something up and the toolbar buttons related to the Page: namespace are no longer coming up.

Compare...

• https://test2.wikipedia.org/w/index.php?title=Page:The_book_of_try_and_learn.djvu/3&action=edit

with...

• https://en.wikisource.org/w/index.php?title=Page:The_book_of_try_and_learn.djvu/3&action=edit

...to see what I mean.

Scratch Comment 7 - it seems the test2 testbed is not completely configured for the ProofReading extension as a default and the issue lies there. Sorry.

(In reply to George Orwell III from comment #7)

Copying from https://gerrit.wikimedia.org/r/#/c/141296/ :
I was worried that WikiEditor customizations like [
https://en.wiktionary.org/wiki/MediaWiki:Gadget-DeveloperEditorTweaks.js and https://github.com/he7d3r/mw-gadget-CodingToolbar
] will stop working in every wiki were CodeEditor is not used (because in this case there will be no WikiEditor on JS pages). Fortunately, I just checked and it won't break as long as the code uses "mw.loader.using( 'ext.wikiEditor.toolbar', ... )" to add WikiEditor before interacting with it.

I think the same applies to the ProofreadPage extension. It seems the extension can force the toolbar to be loaded on any page it wants by just loading the appropriate RL module.

However I think it makes sense to allow extensions to configure the list of content models against which $editPage->contentModel is tested. That is, something like
if ( in_array( $editPage->contentModel, array( CONTENT_MODEL_WIKITEXT, ... ) ) ) {
instead of
if ( $editPage->contentModel !== CONTENT_MODEL_WIKITEXT ) {
where "..." could be set by other extensions which rely on WikiEditor.

All patches mentioned in this report were merged or abandoned - is there more work left to do here (if yes: please reset the bug report status to NEW or ASSIGNED), or can you close this ticket as RESOLVED FIXED?

• Bug 24713 has been marked as a duplicate of this bug. ***