Page MenuHomePhabricator

Issue of TwoColConflict with new Wikitext Editor
Closed, ResolvedPublic

Description

There is an issue when using TwoColConflict together with the new Wikitext editor. Both extensions are available as beat-features on beta, so the issue can be reproduced there by enabling both beta-features and going through a normal conflict resolution.

When a conflict occurs, a dialog is shown:

2cc_we17_conflict_notice.jpg (947×2 px, 112 KB)

After clicking "resolve conflict" the TwoColConflict resolution-screen is shown including a new icon to switch back to visual editing:

2cc_we17_conflict_resolution.jpg (1×2 px, 277 KB)

After clicking the new icon, another dialog asks you again if you really want to switch back to visual editing (everything is fine until here):

2cc_we17_conflict_switch_to_ve.jpg (1×2 px, 214 KB)

Only after confirming the switch, the visual Wikitext editor loads again and the page is screwed up:

2cc_we17_conflict_issue_after_switching_back.jpg (1×3 px, 178 KB)

Event Timeline

We have a similar hack for the translation extension but really we should have an API for this. (ve.init.mw.DesktopArticleTarget.init.js, line 773)

@Esanders - so for now, we could also set some global JS variable and add a line for it there? :-)

Change 335017 had a related patch set uploaded (by WMDE-Fisch):
Disable VisualEditor in the TwoColConflict extension

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

Change 335018 had a related patch set uploaded (by WMDE-Fisch):
Set global JS variable to disable VE

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

Change 335017 merged by jenkins-bot:
Disable VisualEditor in the TwoColConflict extension

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

Change 335018 merged by jenkins-bot:
Set global JS variable to disable VE

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

Change 335025 had a related patch set uploaded (by WMDE-Fisch):
Only deactivte the switch button in the TwoColumnConflict extension

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

Tobi_WMDE_SW moved this task from Done to Demoed on the WMDE-TechWish board.

Change 335025 abandoned by WMDE-Fisch:
Only deactivate the switch button in the TwoColumnConflict extension

Reason:
See comments.

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