Opened 14 years ago
Closed 14 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 14 years ago by
Component: | General → Core : Styles |
---|---|
Status: | new → confirmed |
Changed 14 years ago by
Attachment: | 6774.patch added |
---|
comment:2 Changed 14 years ago by
Owner: | set to Alfonso Martínez de Lizarrondo |
---|---|
Status: | confirmed → review |
comment:3 Changed 14 years ago by
Status: | review → review_failed |
---|
For me the rest of content.css are subjected to move.
comment:4 Changed 14 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 14 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 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [6685]
Proposed patch