Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

T/61: Backspace on a fully selected content or at the beginning of the first block should ensure that paragraph is left #106

Closed
wants to merge 2 commits into from

Conversation

pomek
Copy link
Member

@pomek pomek commented Jul 5, 2017

Suggested merge commit message (convention)

Fix: Backspace on a fully selected content or at the beginning of the first block will ensure that paragraph is left. Closes ckeditor/ckeditor5#3065.

@pomek pomek requested a review from Reinmar July 5, 2017 10:35
@@ -175,7 +176,7 @@ describe( 'InputCommand integration', () => {
'<p>Foo1</p><p>Bar2{}</p>' );
} );

it( 'resets the buffer when attribute changes', () => {
xit( 'resets the buffer when attribute changes', () => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

… first block will ensure that paragraph is left.
@Reinmar
Copy link
Member

Reinmar commented Jul 5, 2017

And what about backspace at the beginning of the first block?

@Reinmar
Copy link
Member

Reinmar commented Jul 5, 2017

A case to check: whether attributes of the block in which the selection is located initially don't leak to the paragraph created afterwards.

@Reinmar
Copy link
Member

Reinmar commented Jul 6, 2017

Let's remove this PR because it still requires some thinking what we want to achieve and some major changes in the implementation.

@Reinmar Reinmar closed this Jul 6, 2017
@pomek pomek deleted the t/61 branch July 6, 2017 09:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants