Steps to reproduce:
- Open VE
- Make an edit
- Reload the editor with unsaved changes
- Click on "Publish Changes"
Observe that the labels for Minor edit and Watch page checkboxes are missing on the save page dialog.
Ryasmeen | |
Mar 2 2018, 11:37 PM |
F16004471: image.png | |
Mar 23 2018, 7:56 PM |
F14332384: Screen Shot 2018-03-02 at 2.38.33 PM.png | |
Mar 2 2018, 11:37 PM |
Steps to reproduce:
Observe that the labels for Minor edit and Watch page checkboxes are missing on the save page dialog.
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Remember 'checkboxesMessages' when storing document state | mediawiki/extensions/VisualEditor | master | +2 -0 |
Change 419955 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] Remember 'checkboxesMessages' when storing document state
Change 419955 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Remember 'checkboxesMessages' when storing document state
Still present (on English Wikipedia, which is on wmf/1.31.0-wmf.26, which is shown to have that commit).
Changes were recovered in the editor.
But the change might have been saved before the fix (though it would be good to do at least 'watchthis' and 'minoredit' from core)
@Mattflaschen-Personal Right, the text of the messages to use is saved in session storage, together with the document and changes to recover, so they would still be missing for changes autosaved before we the fix was deployed. This is a bit unfortunate, but since session storage (and autosaved changes) is cleared anyway when closing the browser tab/window, I think this has already become a non-issue after a few days have passed since the deployment.