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

T/ckeditor5/1151: Brought support for RTL content in the bindTwoStepCaretToAttribute() helper #1769

Merged
merged 12 commits into from
Aug 12, 2019

Conversation

oleq
Copy link
Member

@oleq oleq commented Jul 30, 2019

Suggested merge commit message (convention)

Feature: Brought support for RTL content in the bindTwoStepCaretToAttribute() helper. See ckeditor/ckeditor5#1151.

BREAKING CHANGE: The bindTwoStepCaretToAttribute() helper arguments syntax has changed (replaced by an object). Please refer to the helper documentation to learn more.


Additional information

A piece of ckeditor/ckeditor5#1881.

@coveralls
Copy link

coveralls commented Jul 30, 2019

Coverage Status

Coverage decreased (-0.3%) to 99.652% when pulling 99b2458 on t/ckeditor5/1151 into c057a94 on master.

@oleq oleq added the pr:sub label Jul 30, 2019
@oleq oleq marked this pull request as ready for review August 5, 2019 14:18
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.

Some implementation details could be improved. Other than that, works fine.

@Reinmar Reinmar self-assigned this Aug 7, 2019
@Reinmar Reinmar merged commit d57ff5a into master Aug 12, 2019
@Reinmar Reinmar deleted the t/ckeditor5/1151 branch August 12, 2019 10:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants