| Juandev | |
| Feb 1 2017, 6:34 AM |
| F5443848: pasted_file | |
| Feb 1 2017, 5:20 PM |
| F5443839: pasted_file | |
| Feb 1 2017, 5:20 PM |
Steps to reproduce. In wikitext editor 2016. On this site, I click after edding some images (see revision) on Save changes, pop up window comes, I click See changes, I click Save changes, pup up window comes, but inmediatelly disapper, because page is safed. This is unexpected behaviour as normally you can see preview and than you can also add edit summary.
Experienced on cs an pl Wikipedia. FF 51.0.1
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| MWSaveDialog: Switch to save panel when showing message | mediawiki/extensions/VisualEditor | master | +2 -1 |
I am sorry for the tags, I cannot edit them, when creating task and I can change them even now.
I think we can rephrase this as: in the Save dialog, the "save changes" button in the top-right should only trigger an actual save when we're on the save panel; on the diff / preview panels, it should instead return us to the save panel.
This functionality does already exist, in the form of the "return to save panel" button at the bottom of the dialog, but we could argue that it's unexpected behavior.
If nothing else, if we preserve the current behavior, we probably shouldn't flicker up the save panel like that.
Also, it's not a wikitext editor bug. It's just a general save dialog bug -- note that viewing the diff in regular VE has the same behavior.
I dont know if the pop up window has any name. Is there a way to get this information? Anyway the one to edit revission summary and safe changes.
Well looking the diff probably so, but looking the preview not. You can see preview in VisualEditor, than click on save, enter comments for revision and save. Here once you want to see preview, you have no time to enter revision description.
So… this is behaving as designed. There was a request for users who want to go fast for a button to immediately save whilst looking at the diff (or preview). If you want to be prompted to add an edit summary before save, switching on this option will prompt you in these circumstances. Not sure what to do here.
@Jdforrester-WMF: I do think that stopping the flash of the main save pane would be a worthwhile thing -- it gives an impression of "I could have edited this if only I was quicker" which is misleading.
Beyond that... maybe just making the "go to summary" button more visible via colors? Or moving it to the top-right next to the "save immediately" button?
Change 338134 had a related patch set uploaded (by DLynch):
MWSaveDialog: Switch to save panel when showing message
Change 338134 merged by jenkins-bot:
MWSaveDialog: Switch to save panel when showing message
Yeah, that should help a bit. Our design policy means only one primary (solid-colour) button, so that's out.
More broadly, we should address T44138: VisualEditor: Toolbar "Save page" button is confusing as it merely opens the dialog to save the page.