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

Fix (indent): blockIndent now works with custom headings. Closes #8177. #8459

Merged
merged 4 commits into from
Nov 19, 2020

Conversation

antoniolucasnobar
Copy link

Fix (indent): blockIndent now works with custom headings. Closes #8177.


Additional information

Please let me know if you want me to change anything.

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.

Hi @antoniolucasnobar, thanks for a great PR!

We really appreciate PRs with tests :)

I've added some additions to the PR just to have those covered in the manual tests - mostly used by the CKEditor 5 team.

I was thinking about removing the || since the configuration option should be always present but we do allow implementing the own heading plugin, so in theory, it could be that heading.options will not be defined.

Big thanks!

ps.: Bonus points for using fancy heading as it was my favorite addition to the docs :D

@jodator
Copy link
Contributor

jodator commented Nov 19, 2020

BTW, I've fixed the headings demo: 12b1732.

@jodator jodator merged commit 23c64f5 into ckeditor:master Nov 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not possible to use Block Indent Feature with custom headings
3 participants