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

Internal: Changed converters after engine/conversion refactoring. #91

Merged
merged 3 commits into from
Jan 31, 2018

Conversation

oskarwrobel
Copy link
Contributor

Suggested merge commit message (convention)

Internal: Changed converters after engine/conversion refactoring. Closes https://github.com/ckeditor/ckeditor5-engine/issues/1213.


Part of: ckeditor/ckeditor5-engine#1256

* List structure after the middle element removed:
*
* <listItem type="bulleted" indent=0>Item 1</listItem>
* <listItem type="bulleted" indent=2>Item 3</listItem>
Copy link

@pjasiun pjasiun Jan 30, 2018

Choose a reason for hiding this comment

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

Indent problem (ironically).

@@ -352,58 +353,44 @@ export function modelViewMergeAfter( evt, data, conversionApi ) {
export function viewModelConverter( evt, data, consumable, conversionApi ) {
if ( consumable.consume( data.input, { name: true } ) ) {
const writer = conversionApi.writer;
const conversionData = this.conversionApi.data;
Copy link

@pjasiun pjasiun Jan 30, 2018

Choose a reason for hiding this comment

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

@coveralls
Copy link

coveralls commented Jan 30, 2018

Coverage Status

Coverage decreased (-12.2%) to 87.78% when pulling 6cc7b03 on t/ckeditor5-engine/1213 into 804f85d on master.

@oskarwrobel
Copy link
Contributor Author

oskarwrobel commented Jan 31, 2018

Everything is fixed.

@pjasiun pjasiun merged commit 2523044 into master Jan 31, 2018
@pjasiun pjasiun deleted the t/ckeditor5-engine/1213 branch January 31, 2018 15:21
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