Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added data-cke attribute to style selector #609

Merged
merged 2 commits into from
Apr 4, 2020
Merged

Added data-cke attribute to style selector #609

merged 2 commits into from
Apr 4, 2020

Conversation

pomek
Copy link
Member

@pomek pomek commented Apr 1, 2020

Suggested merge commit message (convention)

Other: The inline style element which is being produced by Webpack and delivers styles for the editor will have the [data-cke="true"] attribute in order to help find CKEditor 5 styles. See ckeditor/ckeditor5#6454.

…and delivers styles for the editor will have the `[data-cke="true"]` attribute in order to help find CKEditor 5 styles. See ckeditor/ckeditor5#6454.
@Reinmar
Copy link
Member

Reinmar commented Apr 1, 2020

CI failed for some reason.

@pomek
Copy link
Member Author

pomek commented Apr 1, 2020

Fails because one of the dependencies we use requires Node 10 => dotansimha/graphql-code-generator#3412.

It's time for upgrading node version on CI.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 91.604% when pulling da99ef5 on i/6454 into 65ae0ff on master.

@pomek
Copy link
Member Author

pomek commented Apr 3, 2020

CI is green again.

@Reinmar Reinmar merged commit 223df34 into master Apr 4, 2020
@Reinmar Reinmar deleted the i/6454 branch April 4, 2020 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants