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.

Sign up for GitHub

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

Jump to bottom

Providing skin path in config breaks balloonpanel css loading #1221

Closed
wojtekw92 opened this issue Nov 24, 2017 · 0 comments · Fixed by #1222
Closed

Providing skin path in config breaks balloonpanel css loading #1221

wojtekw92 opened this issue Nov 24, 2017 · 0 comments · Fixed by #1222
Labels
status:confirmed An issue confirmed by the development team. target:minor Any docs related issue that can be merged into a master or major branch. type:bug A bug.
Milestone
4.8.0

Comments

@wojtekw92
Copy link
Contributor

wojtekw92 commented Nov 24, 2017 •
edited by mlewand
Loading

Are you reporting a feature request or a bug?

Bug

Provide detailed reproduction steps (if any)

When you try to use skin's path in config: https://docs.ckeditor.com/ckeditor4/docs/#!/api/CKEDITOR.config-cfg-skin, inline toolbar will not be able to load proper skin css, even if those are present in library.

  1. Add path to CSS config
  2. Run balloon panel

Expected result

Balloon panel is loading without CSS errors.

Actual result

CSS link is broken.
@wojtekw92 wojtekw92 self-assigned this Nov 24, 2017
@wojtekw92 wojtekw92 added status:confirmed An issue confirmed by the development team. type:bug A bug. labels Nov 24, 2017
@wojtekw92 wojtekw92 mentioned this issue Nov 24, 2017
Merged
2 tasks
@mlewand mlewand added this to the Backlog milestone Nov 24, 2017
@mlewand mlewand added the target:minor Any docs related issue that can be merged into a master or major branch. label Nov 24, 2017
@wojtekw92 wojtekw92 mentioned this issue Nov 24, 2017
Merged
2 tasks
@mlewand mlewand modified the milestones: Backlog, 4.8.0 Nov 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
status:confirmed An issue confirmed by the development team. target:minor Any docs related issue that can be merged into a master or major branch. type:bug A bug.
Projects
None yet
Milestone
4.8.0
Development

Successfully merging a pull request may close this issue.

CSS loading fix in balloonpanel ckeditor/ckeditor4
2 participants
@mlewand @wojtekw92