Ticket #5577 (closed defect (bug): fixed)

Opened 4 years ago

Last modified 4 years ago

Switching between HTML and visiual views destroys some table markup

Reported by: mdawaffe Owned by: 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 Download (1.8 KB) - added by mdawaffe 4 years ago.

Change History

  • Keywords has-patch commit added

comment:2   ryan4 years ago

  • 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

Note: See TracTickets for help on using tickets.