Opened 15 years ago
Closed 15 years ago
#6774 closed Bug (fixed)
Default contents.css should provide only sample styles
| Reported by: | Alfonso Martínez de Lizarrondo | Owned by: | Alfonso Martínez de Lizarrondo |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.6 |
| Component: | General | Version: | |
| Keywords: | Cc: |
Description
Currently there are some internal rules added there that are lost as soon as the editor is integrated into any CMS and a new config.contentsCss is specified
html
{
/* #3658: [IE6] Editor document has horizontal scrollbar on long lines
To prevent this misbehavior, we show the scrollbar always */
_overflow-y: scroll;
/* #6341: The text cursor must be set on the editor area. */
cursor: text;
/* #6632: Avoid having "text" shape of cursor in IE7 scrollbars.*/
*cursor:auto;
}
img:-moz-broken
{
-moz-force-broken-image-icon : 1;
width : 24px;
height : 24px;
}
img, input, textarea
{
cursor: default;
}
All these rules should be created inline by the plugins (and even included only for the correct browsers).
Attachments (1)
Change History (7)
comment:1 Changed 15 years ago by
| Component: | General → Core : Styles |
|---|---|
| Status: | new → confirmed |
Changed 15 years ago by
| Attachment: | 6774.patch added |
|---|
comment:2 Changed 15 years ago by
| Owner: | set to Alfonso Martínez de Lizarrondo |
|---|---|
| Status: | confirmed → review |
comment:3 Changed 15 years ago by
| Status: | review → review_failed |
|---|
For me the rest of content.css are subjected to move.
comment:4 Changed 15 years ago by
| Status: | review_failed → review |
|---|
If we remove the rest of the content.css then the provided samples would show up in Times New Roman unless we create a new "contentSample.css" file to put there some basic styles like the content.css file has now, so I don't see how that's related to the issue handled in this ticket.
comment:5 Changed 15 years ago by
| Component: | Core : Styles → General |
|---|---|
| Milestone: | → CKEditor 3.6 |
| Status: | review → review_passed |
Please be sure to have it committed into the 3.6.x branch.
comment:6 Changed 15 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [6685]

Proposed patch