Opened 13 years ago

Closed 13 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)

6774.patch (1.8 KB) - added by Alfonso Martínez de Lizarrondo 13 years ago.
Proposed patch

Download all attachments as: .zip

Change History (7)

comment:1 Changed 13 years ago by Garry Yao

Component: GeneralCore : Styles
Status: newconfirmed

Changed 13 years ago by Alfonso Martínez de Lizarrondo

Attachment: 6774.patch added

Proposed patch

comment:2 Changed 13 years ago by Alfonso Martínez de Lizarrondo

Owner: set to Alfonso Martínez de Lizarrondo
Status: confirmedreview

comment:3 Changed 13 years ago by Garry Yao

Status: reviewreview_failed

For me the rest of content.css are subjected to move.

comment:4 Changed 13 years ago by Alfonso Martínez de Lizarrondo

Status: review_failedreview

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 13 years ago by Frederico Caldeira Knabben

Component: Core : StylesGeneral
Milestone: CKEditor 3.6
Status: reviewreview_passed

Please be sure to have it committed into the 3.6.x branch.

comment:6 Changed 13 years ago by Alfonso Martínez de Lizarrondo

Resolution: fixed
Status: review_passedclosed

Fixed with [6685]

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy