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

The Justify plugin doesn't obey the custom Advanced Content Filter rules #455

Closed
Comandeer opened this issue Jun 2, 2017 · 0 comments
Closed
Assignees
Labels
status:confirmed An issue confirmed by the development team. type:bug A bug.
Milestone

Comments

@Comandeer
Copy link
Member

Are you reporting a feature or a bug?

Bug

Check if the issue is already reported

http://dev.ckeditor.com/ticket/16808

Provide detailed reproduction steps (if any)

  1. Make sure the Justify plugin is available.
  2. Configure the editor to use the following custom ACF rules:
allowedContent: 'p ul{text-align};li'
  1. Load the editor. The Justify tool bar buttons are available, which is expected because we have the "p ul{text-align}" rule.
  2. Verify that the custom ACF rules are active using the JavaScript console:
    5 .Create a list
  3. Place the caret on the second item. The Justify buttons are still enabled. Click on the "Center" button from the tool bar.

Expected result

Justify plugin should prevent of applying justification on disallow elements. If ACF forbid to apply text-align style on given element, then plugin should be disabled on such element.

Actual result

The Justify plugin centers the second list item, thus generating invalid content from the point of view of the ACF.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:confirmed An issue confirmed by the development team. type:bug A bug.
Projects
None yet
Development

No branches or pull requests

3 participants