Last modified: 2014-11-19 23:38:32 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 T73706, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 71706 - VisualEditor: [Regression wmf5] Comment inspector closes and saves empty comment when tool is used
VisualEditor: [Regression wmf5] Comment inspector closes and saves empty comm...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: Normal minor
: VE-deploy-2014-11-19 (1.25wmf9)
Assigned To: Ed Sanders
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-10-06 14:54 UTC by Ritu Swain
Modified: 2014-11-19 23:38 UTC (History)
9 users (show)

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


Attachments
Screenshot (124.13 KB, image/png)
2014-10-06 14:54 UTC, Ritu Swain
Details

Description Ritu Swain 2014-10-06 14:54:36 UTC
Created attachment 16677 [details]
Screenshot

Environment- test2, beta, production

1> insert a comment in a blank page.
2> open the Edit comment box
3> while the cursor is still blinking in the comment box, click to add numbered / Bullet point. The comment box closes and the comment is shown as the bullet/numbered list item
4> Now open the edit comment box again and decrease the indentation.I get "JS console error, Uncaught Error: Cannot set attributes on closing element".See screenshot.
Comment 1 Roan Kattouw 2014-10-22 02:03:30 UTC
This isn't as bad any more (no JS errors), probably thanks to Ed's change making the comment inspector use staging. But there's still a minor bug:

* Insert a comment in a blank page
* Type some text into the comment
* While comment box is still open, click Bullet list
* Comment inspector closes, comment ends up inside list
* Inspect comment
* Comment is empty (?!)
* Close inspector. Comment is automatically removed because it is empty

(If you don't type some text at step 2, the comment is removed at that point because it's empty.)

(Marking as regression pre-wmf5 even though the behavior in wmf4 was much worse.)
Comment 2 Gerrit Notification Bot 2014-11-17 13:54:02 UTC
Change 173812 had a related patch set uploaded by Esanders:
Apply comment changes using fragment

https://gerrit.wikimedia.org/r/173812
Comment 3 Gerrit Notification Bot 2014-11-17 18:16:00 UTC
Change 173812 merged by jenkins-bot:
Apply comment changes using fragment

https://gerrit.wikimedia.org/r/173812
Comment 4 etonkovidova 2014-11-19 23:38:32 UTC
test2 - checked both scenarios - no js errors.

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


Navigation
Links