Last modified: 2014-07-01 16:29:01 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T69345, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 67345 - VisualEditor: Prompt for edit summary preference breaks the save dialog
VisualEditor: Prompt for edit summary preference breaks the save dialog
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
MediaWiki integration (Other open bugs)
unspecified
All All
: Low normal
: VE-deploy-2014-07-03
Assigned To: Alex Monk
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-07-01 03:39 UTC by Keegan Peterzell
Modified: 2014-07-01 16:29 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Keegan Peterzell 2014-07-01 03:39:45 UTC
If a user has the preference "Prompt me when entering a blank edit summary" selected, VisualEditor's save dialog has to be closed and reopened to work. The "Save" button is greyed out even when an edit summary is entered.

To reproduce:
1. Enable Preferences>Editing>tick Prompt me when entering a blank edit summary
2. Attempt to save a VE edit and the prompt for an edit summary comes up, with save not an option
3. Enter an edit summary
4. Save is still not an option
5. Exit out of the dialog by clicking X and click save again
6. You can now save your edit with an edit summary

Firefox 30 on Mac OS X Mavericks.
Comment 1 James Forrester 2014-07-01 04:01:42 UTC
Sounds like we need to code in that the you-must-set-an-edit-summary message is a recoverable error.
Comment 2 Gerrit Notification Bot 2014-07-01 14:54:21 UTC
Change 143326 had a related patch set uploaded by Alex Monk:
After showing missing edit summary error, mark the dialog as not pending anymore

https://gerrit.wikimedia.org/r/143326
Comment 3 Alex Monk 2014-07-01 14:57:32 UTC
This was probably an oversight on my part in Gerrit change #135945
Comment 4 Gerrit Notification Bot 2014-07-01 16:29:01 UTC
Change 143326 merged by jenkins-bot:
After showing missing edit summary error, mark the dialog as not pending anymore

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

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links