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

Soft break inside list element will be handled correctly. #117

Merged
merged 4 commits into from
Nov 13, 2018

Conversation

pomek
Copy link
Member

@pomek pomek commented Nov 9, 2018

Suggested merge commit message (convention)

Fix: Block filler will be inserted into the list item if its last child is a <br> element. Closes ckeditor/ckeditor5#1312.

@coveralls
Copy link

coveralls commented Nov 9, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling 9dfbffe on t/ckeditor5/1312 into bfda8b2 on master.

@Reinmar Reinmar self-requested a review November 13, 2018 12:45
Copy link
Member

@Reinmar Reinmar left a comment

Choose a reason for hiding this comment

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

Couldn't we reuse the logic of the original getFillerOffset()? Isn't this function just an extension of the original one?

@Reinmar Reinmar removed their assignment Nov 13, 2018
@pomek
Copy link
Member Author

pomek commented Nov 13, 2018

Done. This PR requires ckeditor/ckeditor5-engine#1591 now.

image

Reinmar added a commit to ckeditor/ckeditor5-engine that referenced this pull request Nov 13, 2018
Other: `ContainerElement#getFillerOffset` can be now re-used in other places in the code (it was exported from the module). See ckeditor/ckeditor5-list#117.
Co-Authored-By: pomek <pomek@users.noreply.github.com>
@Reinmar Reinmar merged commit cb6708e into master Nov 13, 2018
@Reinmar Reinmar deleted the t/ckeditor5/1312 branch November 13, 2018 16:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants