Cannot delete paragraph before and after a widget

[Wiktor Walc]

Found in Chrome, confirmed also on Firefox.

1. Create a table
2. Insert a widget (e.g. Enhanced Image) in a table cell (note: it works only in CKEditor 4.4.1+ #11798)
3. See that below widget an empty paragraph is created. If this additional space is disturbing for somebody, one cannot remove it.

This issue can also be reproduced directly in body: ​http://stackoverflow.com/questions/30262191/ckeditor-how-to-delete-empty-paragraphs-before-and-between-widgets

To reproduce try this code:

<table border="1" cellpadding="1" cellspacing="1" style="width:500px">
	<tbody>
		<tr>
			<td>&nbsp;</td>
			<td>
			<figure class="image"><img alt="dfgdfg" height="480" src="assets/image1.jpg" width="377" />
			<figcaption>Caption</figcaption>
			</figure>

			<p>&nbsp;</p>
			</td>
		</tr>
		<tr>
			<td>&nbsp;</td>
			<td>&nbsp;</td>
		</tr>
	</tbody>
</table>

---

There is one interesting method to actually remove the element. But this is so counter-intuitive that it should not be considered as a correct solution.

1. Put the cursor in the <p> element that you want to removed

2. Select p in elements path (see that additional br showed up in the elements path automatically)

3. Press Backspace. Now the paragraph is gone (although slightly different visual result is in Chrome and Firefox).

[Niklas]

I cannot delete the first <p>&nbsp;</p> if it's followed by <figure>. I tried this with Chrome.

<p>&nbsp;</p>

<figure class="align-left"><img class="img-responsive img-rounded" src="//placehold.it/100" /></figure>

<p>Lorem ipsum!</p>

[Piotrek Koszuliński]

This would require some additional rules for delete and backspace, so if the current paragraph is already empty they should remove it instead of focusing widget. This code could land e.g. here: ​https://github.com/ckeditor/ckeditor-dev/blob/3a10930/core/selection.js#L339-L365.

[Piotrek Koszuliński]

If anyone wants to contribute a patch, then:

• here are tests for the existing behaviour - ​https://github.com/ckeditor/ckeditor-dev/blob/3a109301b738d9998f132b8030d1b1dbe507c92f/tests/core/selection/fake.js#L874-L906
• here is a brand new contributing code guide - ​http://docs.ckeditor.com/#!/guide/dev_contributing_code

[Jakub Ś]

#13864 was marked as duplicate.