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

I/6110: Deprecate LabeledInputView #549

Merged
merged 4 commits into from
Mar 26, 2020
Merged

Conversation

panr
Copy link
Contributor

@panr panr commented Mar 23, 2020

Suggested merge commit message (convention)

Other: LabeledInputView component has been marked as deprecated in favour of use LabeledFieldView component instead. See ckeditor/ckeditor5#6110.


Additional information

This is a standalone PR, related to the changes in:

BREAKING CHANGES:

  • LabeledInputView will be removed soon (probably in version 20.0.0).

@panr panr changed the base branch from master to i/6110 March 23, 2020 16:22
@coveralls
Copy link

coveralls commented Mar 23, 2020

Coverage Status

Coverage remained the same at 100.0% when pulling bc063b9 on i/6110-deprecate-labeledinputview into c5f7e82 on i/6110.

Copy link
Contributor

@jodator jodator left a comment

Choose a reason for hiding this comment

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

LGTM waiting for related PR to merge.

Other: Rename `LabeledView` component to `LabeledFieldView`. See ckeditor/ckeditor5#6110.

BREAKING CHANGE: `LabeledView` component was renamed to `LabeledFieldView`. Also, its instance of a labeled component's view is available through `LabeledFieldView#fieldView`. It replaced `LabeledView#view`.
@jodator jodator merged commit ae08076 into i/6110 Mar 26, 2020
@jodator jodator deleted the i/6110-deprecate-labeledinputview branch March 26, 2020 07:54
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