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.)

gerritadmin wrote:

Change 173812 had a related patch set uploaded by Esanders:
Apply comment changes using fragment

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

gerritadmin wrote:

Change 173812 merged by jenkins-bot:
Apply comment changes using fragment

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

test2 - checked both scenarios - no js errors.