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

Editor UI classes API refactoring #77

Merged
merged 15 commits into from
Jan 22, 2019
Merged

Editor UI classes API refactoring #77

merged 15 commits into from
Jan 22, 2019

Conversation

f1ames
Copy link
Contributor

@f1ames f1ames commented Jan 15, 2019

Suggested merge commit message (convention)

Other: Editor UI classes API refactoring.

BREAKING CHANGE: Removed ClassicEditor#element property. The ClassicEditorUI#element property should be used instead.
BREAKING CHANGE: Removed ClassicEditorUIView#editableElement. Instead ClassicEditorUI#getEditableElement() method should be used.


Additional information

See ckeditor/ckeditor5#1449 and ckeditor/ckeditor5#1449 (comment) describing changes in each commit.

The CI should turn green after ckeditor/ckeditor5-core#154 is merged.

@coveralls
Copy link

coveralls commented Jan 15, 2019

Coverage Status

Coverage decreased (-71.4%) to 28.571% when pulling 07728a5 on t/ckeditor5/1449 into a3c5c82 on master.

src/classiceditor.js Outdated Show resolved Hide resolved
src/classiceditor.js Outdated Show resolved Hide resolved
src/classiceditoruiview.js Outdated Show resolved Hide resolved
*/
get view() {
return this._view;
}
Copy link

@pjasiun pjasiun Jan 18, 2019

Choose a reason for hiding this comment

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

Does we need this public getter?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

src/classiceditor.js Outdated Show resolved Hide resolved
Co-Authored-By: f1ames <dr.odpowiedz@gmail.com>
@f1ames f1ames requested a review from pjasiun January 21, 2019 15:26
@f1ames
Copy link
Contributor Author

f1ames commented Jan 22, 2019

Make sure to rerun CI for this PR after merging ckeditor/ckeditor5-core#154 and ckeditor/ckeditor5-ui#458.

@pjasiun pjasiun merged commit 74e27ae into master Jan 22, 2019
@pjasiun pjasiun deleted the t/ckeditor5/1449 branch January 22, 2019 16:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants