Opened 10 years ago

Last modified 4 years ago

#3140 confirmed Bug

Indent problem with whole table

Reported by: Garry Yao Owned by:
Priority: Normal Milestone:
Component: General Version: 3.0 Beta 2
Keywords: Support Cc:

Description (last modified by Garry Yao)

Currently it's been unable to apply indent to the whole table.

  1. Make the content and selection as below:
    ^<table border="1" cellpadding="1" cellspacing="1" style="width: 200px;">
    	<tbody>
    		<tr>
    			<td>
    				<br />
    			</td>
    		</tr>
    	</tbody>
    </table>^
    
  1. Apply the Indent command;
  • Expected Result:
    <p style="margin-left: 40px;">
    <table border="1" cellpadding="1" cellspacing="1" style="width: 200px;">
    	<tbody>
    		<tr>
    			<td>
    				<p>
    				</p>
    			</td>
    		</tr>
    	</tbody>
    </table>
    </p>
    
  • Actual Result:
    <table border="1" cellpadding="1" cellspacing="1" style="width: 200px;">
    	<tbody>
    		<tr>
    			<td>
    				<p>
    				</p>
    			</td>
    		</tr>
    	</tbody>
    </table>
    <p style="margin-left: 40px;">
    </p>
    

Change History (10)

comment:1 Changed 10 years ago by Garry Yao

Description: modified (diff)
Summary: Table level operation problemIndent problem with whole table

comment:2 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.0CKEditor 3.x

The ticket makes sense, but the exemplified "expected results" is wrong. You can't have <table> inside <p>. The correct result is the following instead:

<table border="1" cellpadding="1" cellspacing="1" style="width: 200px; margin-left: 40px;">
	<tbody>
		<tr>
			<td>
				<br>
			</td>
		</tr>
	</tbody>
</table>

comment:3 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Confirmed added

comment:4 Changed 10 years ago by Alfonso Martínez de Lizarrondo

Version: SVN (FCKeditor)CKEditor 3.0 Beta 2

comment:5 Changed 9 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.x

Milestone CKEditor 3.x deleted

comment:6 Changed 8 years ago by Garry Yao

Covered by #7845

comment:7 Changed 5 years ago by Jakub Ś

This issue is related to #4903.

When selecting every cell in table I would rather expect to indent cells thus perhaps when clicking on table element in element's path we should allow indenting whole table.

I'm not really sure in what other way to allow indenting whole table. Any else has any ideas?

comment:8 Changed 5 years ago by Jakub Ś

Perhaps there is a better idea - introducing new plugin with separate toolbar button and context menu option?
Current behaviour could be left as it is and with this plugin user could indent table by simply clicking on it (or selecting it). Plugin would then search for nearest table tag and apply indentation style there.

comment:9 Changed 5 years ago by Wiktor Walc

Keywords: Support added

comment:10 Changed 4 years ago by Jakub Ś

Related issue or duplicate - #13102.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy