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

Introduced multiple fixes in OT algorithms and position transformation functions #1490

Closed
wants to merge 11 commits into from

Conversation

scofalik
Copy link
Contributor

@scofalik scofalik commented Aug 1, 2018

Suggested merge commit message (convention)

Fix: Introduced multiple fixes in OT algorithms and position transformation functions. Closes ckeditor/ckeditor5#4390.


Additional information

Bases on #1478 .

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 98.46% when pulling d7b475c on t/1475 into 00fbf7f on master.

return ranges;
}

function _flatMoveTransform( ranges, moveOp ) {
Copy link

Choose a reason for hiding this comment

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

Comments needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Spoiler alert: This function is rewritten on branch t/1488 which contains following improvements in OT. Also, I will complete all the docs and also inline comments in another followup.

@pjasiun pjasiun self-requested a review August 9, 2018 10:17
@pjasiun
Copy link

pjasiun commented Aug 9, 2018

Moved to #1497.

@pjasiun pjasiun closed this Aug 9, 2018
@pjasiun pjasiun deleted the t/1475 branch August 9, 2018 13:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants