Opened 5 years ago

Closed 5 years ago

#11350 closed Bug (fixed)

Config.contentsCss should be affected by CKEDITOR.getUrl

Reported by: Piotrek Koszuliński Owned by: Piotrek Koszuliński
Priority: Normal Milestone: CKEditor 4.3.2
Component: General Version:
Keywords: Cc:

Description (last modified by Piotrek Koszuliński)

Currently the default value of config.contentsCss is set to CKEDITOR.basePath + 'contents.css', so overriding window.CKEDITOR_GETURL will not affect it.

Pull request: https://github.com/ckeditor/ckeditor-dev/pull/62

Change History (4)

comment:1 Changed 5 years ago by Piotrek Koszuliński

Description: modified (diff)

comment:2 Changed 5 years ago by Piotrek Koszuliński

Owner: set to Piotrek Koszuliński
Status: newreview

Pushed t/11350 on dev and tests. I took the opportunity and wrote tests for contentsCss.

comment:3 Changed 5 years ago by Piotrek Koszuliński

Status: reviewreview_passed

comment:4 Changed 5 years ago by Piotrek Koszuliński

Milestone: CKEditor 4.3.2
Resolution: fixed
Status: review_passedclosed

Fixed with git:0552435 on dev and 638ac9c on tests.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy