Page MenuHomePhabricator

Update CodeMirror integrations in preparation for CodeMirror 6 upgrade
Open, Stalled, Needs TriagePublic

Description

This task tracks changes needed to extensions and skins for compatibility with CodeMirror 6 (T259059).

Deployed at Wikimedia
Others

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
MusikAnimal raised the priority of this task from Medium to Needs Triage.Feb 16 2024, 5:02 PM

Change 1004201 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/extensions/CharInsert@master] ext.charinsert.js: remove special CodeMirror handling

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

Change 1004802 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/extensions/GrowthExperiments@master] HelpPanelLogger: add check for CodeMirror 6 element

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

Change 1004807 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/extensions/ProofreadPage@master] ext.proofreadpage.page.edit.css: fixes for CodeMirror 6 compatibility

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

Change 1005178 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/skins/Timeless@master] Remove unnecessary CodeMirror skin styles

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

Change 1005841 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/extensions/CodeMirror@master] CodeMirror: default to using the same height as the textarea

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

Change 1005178 merged by jenkins-bot:

[mediawiki/skins/Timeless@master] Remove unnecessary CodeMirror skin styles

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

MusikAnimal changed the task status from Open to In Progress.Feb 24 2024, 4:16 AM

This is still in progress but there are patches awaiting review.

Change 1004802 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] HelpPanelLogger: add check for CodeMirror 6 element

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

Change 1005841 merged by jenkins-bot:

[mediawiki/extensions/CodeMirror@master] CodeMirror: default to using the same height as the textarea

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

Change 1004807 merged by jenkins-bot:

[mediawiki/extensions/ProofreadPage@master] ext.proofreadpage.page.edit: fixes for CodeMirror 6 compatibility

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

Change 1007035 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/extensions/CodeMirror@master] CM6: use textarea height unless WikiEditor's ResizingDragBar is present

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

Change 1007747 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/extensions/ProofreadPage@master] extension.json: register 'proofread-page' in CodeMirrorContentModels

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

Change 1007746 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/extensions/CodeMirror@master] Hooks: further limit where CodeMirror RL modules are loaded

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

Change 1007035 merged by jenkins-bot:

[mediawiki/extensions/CodeMirror@master] CM6: use textarea height unless WikiEditor's Realtime Preview is loaded

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

Change 1009608 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/extensions/WikiEditor@master] ResizingDragBar: force CodeMirror to have 100% height

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

Change 1009609 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/extensions/CodeMirror@master] CodeMirrorWikiEditor: Remove hack to fix height in WikiEditor

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

Change 1007747 merged by jenkins-bot:

[mediawiki/extensions/ProofreadPage@master] extension.json: register 'proofread-page' in CodeMirrorContentModels

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

Change 1004201 merged by jenkins-bot:

[mediawiki/extensions/CharInsert@master] ext.charinsert.js: remove special CodeMirror handling

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

Change 1009608 merged by jenkins-bot:

[mediawiki/extensions/WikiEditor@master] ResizingDragBar: force CodeMirror to have 100% height

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

Change 1009609 merged by jenkins-bot:

[mediawiki/extensions/CodeMirror@master] CodeMirrorWikiEditor: Remove hack to fix height in WikiEditor

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

That should be it for WMF-deployed integrations. I'll work on the others in my spare time.

Change #1064101 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/skins/Timeless@master] Make VisualEditor CodeMirror styles target CodeMirror 6

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

Change #1064101 merged by jenkins-bot:

[mediawiki/skins/Timeless@master] Make VisualEditor CodeMirror styles target CodeMirror 6

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

MusikAnimal changed the task status from In Progress to Stalled.Thu, Dec 12, 10:47 PM

Remaining integrations will have to be fixed by respective maintainers.

I'll send an email to wikitech-l soon announcing the deprecations, as part of T373720.