[FF] Unable to remove list with Ctrl-A

[garry.yao]

Reproducing Procedures

1. Open the 'replace by class' sample page in Firefox;
2. Load the document with the following content and selection with Ctrl-A:

   [<p>line1</p>
   <p>line2</p>]
   

3. Click on 'Insert/Remove Bulleted List' button twice;
   • Actual Result: The second click of remove list has no effect.

[garry.yao]

After discussing with Tobias, I'm taking over this ticket.

[tobiasz.cudnik]

Removing list works after second list-command with the patch, but after third one nothing is happening. Expected is that list will be created again.

[garry.yao]

I'm trying to persuade that this patch's not being a drastic change:

1. Changes in previous 4129.patch are leveraged without change.
2. Changes to _source/plugins/wysiwygarea/plugin.js and _source/core/dom/element.js are because it's breaking the 'selectionChange' event

which stop the list command from freshing( Tobias's comment );

3. Changes to _source/plugins/domiterator/plugin.js are necessary clean up to avoid extra list item ( contain only whitespace text ) been created when exec list command multiple times.

[tobiasz.cudnik]

I can confirm that issues i've mentioned is fixed with second patch.

[garry.yao]

Fixed with [4034].