Opened 14 years ago
Closed 14 years ago
#4194 closed New Feature (fixed)
Using multiple css stylesheets within the editor
Reported by: | Sa'ar Zac Elias | Owned by: | Wiktor Walc |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.1 |
Component: | General | Version: | |
Keywords: | Confirmed Review+ | Cc: |
Description
With FCKEditor, we could use several stylesheet within the editor area if using an array of stylesheets' paths instead of a string in the config elemnt.
i think it will be great if we could do this on V3 too.
Attachments (2)
Change History (8)
comment:1 Changed 14 years ago by
Keywords: | Confirmed added |
---|---|
Milestone: | → CKEditor 3.1 |
Changed 14 years ago by
Attachment: | 4194.patch added |
---|
comment:2 Changed 14 years ago by
Keywords: | Review? added |
---|---|
Owner: | set to Wiktor Walc |
comment:3 Changed 14 years ago by
Keywords: | Review- added; Review? removed |
---|
Changed 14 years ago by
Attachment: | 4194_2.patch added |
---|
comment:4 follow-up: 5 Changed 14 years ago by
Keywords: | Review? added; Review- removed |
---|
The newest patch is using simpler method of concatenating suggested by fredck.
There is one little thing that has changed: the order of css files.
Before it was: [contentsCss, editorCss], now it is: [editorCss, contentsCss].
comment:5 Changed 14 years ago by
Keywords: | Review+ added; Review? removed |
---|
Replying to wwalc:
There is one little thing that has changed: the order of css files.
Hum... good point. It should not be an issue for now, but good to keep it in mind.
The patch looks good. But it looks like there would be a simpler way for the concat handling:
The concat() function should simply copy the items from config.contentsCss if it's an Array, otherwise it will be simply copied directly (for a String).
The simplification could be applied to the <link> creation trick also, with no performance impact: