Ticket #5577 (closed defect: fixed)

Opened 8 months ago

Last modified 8 months ago

Switching between HTML and visiual views destroys some table markup

Reported by: mdawaffe Assigned to: anonymous
Priority: normal Milestone: 2.5
Component: General Version:
Severity: normal Keywords: JS has-patch commit
Cc:

Description

To reproduce:

  1. Enter the following into HTML view (formerly known as "Code view").
    <table border="1" width="100%">
    	<tr>
    		<th align="center">Centered</th>
    		<th>Heading 1</th>
    		<th>Heading 2</th>
    	</tr>
    
    	<tr bgcolor="#cccccc">
    		<td><b>Bold</b></td>
    		<td>Foo</td>
    		<td>Bar</td>
    	</tr>
    </table>
    
  1. Switch to Visual view.
  2. Switch back to HTML view.
  3. Notice that the second TR is gone.

This is caused by a bad regex in the JS that converts between the two views.

Patch attached.

Attachments

5577.diff (1.8 kB) - added by mdawaffe on 01/04/08 00:41:18.

Change History

01/04/08 00:41:18 changed by mdawaffe

  • attachment 5577.diff added.

01/04/08 00:42:49 changed by mdawaffe

  • keywords changed from JS to JS has-patch commit.

01/04/08 08:49:35 changed by ryan

  • status changed from new to closed.
  • resolution set to fixed.

(In [6547]) Don't bust table markup when switching between visual and html. Props mdawaffe. fixes #5577