Indent problem with whole table

[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>
  

[Frederico Caldeira Knabben]

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>

[Frederico Caldeira Knabben]

Milestone CKEditor 3.x deleted

[Garry Yao]

Covered by #7845

[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?

[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.

[Jakub Ś]

Related issue or duplicate - #13102.