thanhlong203 / closure-library

Automatically exported from code.google.com/p/closure-library
0 stars 0 forks source link

Remove Formatting plugin add better support for tables #221

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
As of r337, the remove formatting editor plugin performs incorrectly on some 
browsers when the selection spans multiple table cells but does not cover the 
entire table.

http://www.youtube.com/watch?v=qsGkJ1JV1nA

Example:

0) Browse to the editor demo at: 
http://closure-library.googlecode.com/svn/trunk/closure/goog/demos/editor/editor
.html
1) Using the "Set Field Contents" add the following table:

"""<table width="100%" border="1">
<tr><td>a</td><td><font size=6>b</font></td></tr>
</table>"""

2) Using the mouse, click and drag to select "a" and "b".
3) Click the "Remove Formatting" button.
4) CTRL-A to select the entire table.
4) Click "Remove Format" again.

Results:

FF - Works as expected. Formatting is removed and contents remain in their 
table cell. With CTRL-A entire table is removed.

IE - JS error (Unexpected Error). Formatting _is not_ removed. CTRL-A works as 
expected (entire table is removed).

Chrome/Safari - b moves into a's table cell. Formatting is removed. CTRL-A 
works as expected (entire table is removed).

Opera - b moves into a's table cell and b's cell is removed from the table. 
Formatting is removed. CRTL-A _does not_ remove the entire table.

Original issue reported on code.google.com by ahochh...@samegoal.com on 5 Oct 2010 at 8:31

GoogleCodeExporter commented 8 years ago
The JS error in IE is "Unspecified Error" (not "Unexpected Error" as written 
above).

Original comment by ahochh...@samegoal.com on 5 Oct 2010 at 8:41

GoogleCodeExporter commented 8 years ago

Original comment by chrishe...@google.com on 27 Apr 2012 at 9:46