Page MenuHomePhabricator

Feedback dialog loses user input in case of failure
Closed, ResolvedPublic1 Estimated Story Points


From User:Storkk:

Using VE on Commons. I have now twice typed a detailed bug report using VE's own "Help -> Leave feedback about this software", and upon clicking submit get the helpful error message "Error: Unable to post to given feedback title" and lost the entire report. In case of error, the report should be echoed back in the Error dialog box so that at least it can be copied/pasted. The Subject field contained no special characters and wasn't overly long.
Secondly, the Submit Feedback dialog links to "VisualEditor/Feedback" on Commons (ie. - a page which doesn't exist) rather than here.

Event Timeline

Please treat this as a Feature Request to echo back the input text when displaying any Error message.

The initial error appears to be unreproducible, and the second part is T125400.

Easy way to test (on Chromium, other browsers probably have something similar): open the feedback dialog, then open the developer tools and check the "Offline" checkbox in "Network" tab.

pasted_file (1×1 px, 146 KB)

Clicking "Submit" will close the dialog and display an entirely new one:

pasted_file (144×317 px, 4 KB)

Instead, it should keep the dialog open and display the error message in the standard way. For example, the file upload dialog does it right:

pasted_file (1×1 px, 137 KB)

pasted_file (448×511 px, 32 KB)

(well, that error message is far from perfect, but it's still better than closing the dialog; and we have it filed as T132611)

This tool is technically part of MediaWiki core ( and used in some other places, but VE is probably the most prominent (and the original).

matmarex triaged this task as Medium priority.

Might as well fix it, since I already looked into it.

Change 313616 had a related patch set uploaded (by Bartosz Dziewoński): Use standard OOjs UI dialog error handling

With the patch:

pasted_file (1×1 px, 144 KB)

I forgot how nice these errors are, there's even a built-in retry button ;)

matmarex renamed this task from Error: Unable to post to given feedback title to Feedback dialog loses user input in case of failure.Sep 30 2016, 7:52 PM
Jdforrester-WMF set the point value for this task to 1.
Jdforrester-WMF moved this task from To Triage to TR0: Interrupt on the VisualEditor board.

Change 313616 merged by jenkins-bot: Use standard OOjs UI dialog error handling