Opened 8 years ago
Closed 7 years ago
#4194 closed New Feature (fixed)
Using multiple css stylesheets within the editor
| Reported by: | Saare | Owned by: | wwalc |
|---|---|---|---|
| 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 8 years ago by fredck
- Keywords Confirmed added
- Milestone set to CKEditor 3.1
Changed 7 years ago by wwalc
comment:2 Changed 7 years ago by wwalc
- Keywords Review? added
- Owner set to wwalc
comment:3 Changed 7 years ago by fredck
- Keywords Review- added; Review? removed
Changed 7 years ago by wwalc
comment:4 follow-up: ↓ 5 Changed 7 years ago by wwalc
- 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 in reply to: ↑ 4 Changed 7 years ago by fredck
- 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.
comment:6 Changed 7 years ago by wwalc
- Resolution set to fixed
- Status changed from new to closed
Fixed with [4363].
Note: See
TracTickets for help on using
tickets.

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: