Page MenuHomePhabricator

[Regression pre-wmf.2] Switching back from the diff panel to the save form on mobile VE makes the form go empty
Closed, ResolvedPublic

Description

Steps to replicate:

  1. Open mobile VE
  2. Make an Edit and tap on "Publish"
  3. Tap on "Review your changes"
  4. Tap on "Return to save form"

The save form is empty.

IMG_2501.jpg (1×640 px, 50 KB)

Event Timeline

  1. Open mobile VE
  2. Make an Edit and tap on "Publish"
  3. Tap on "Review your changes"
  4. Tap on "Return to save form"

For completeness' sake: I found another step after 3. to be necessary: switch to "Visual" diff if you were viewing a "Wikitext" diff.

Given the above and T219680, there is clearly something in our visual diffs that triggers this Safari bug when we hide them. If anyone fancies investigating it, start here :) I'll just apply the same fix here as in the other task.

Change 505860 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] ve.ui.MWSaveDialog: Work around save panel disappearing on iOS Safari as well

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

Change 505860 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] ve.ui.MWSaveDialog: Work around save panel disappearing on iOS Safari as well

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

@matmarex: So this is not happening anymore when I follow the original steps but while testing the fix I found another case where it seems still broken. The steps are as follows:

Steps:

  1. Make an edit.
  2. Tap on Publish.
  3. Tap on "Review your changes".
  4. After switching to the diff panel, just close it.
  5. Now tap on "Publish" again.

The save panel is empty again.

Change 507704 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] ve.ui.MWSaveDialog: More save panel disappearing on iOS Safari

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

Change 507704 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] ve.ui.MWSaveDialog: More save panel disappearing on iOS Safari

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

Confirming the issue this task was originally written to address, and the issue @Ryasmeen identified here, have both been resolved.

However, in the process of testing in the above, a new issues seems to have emerged: T222489: Visual diffs not showing in mobile VE (when visual section editing is disabled)