Label Style should have float:none

[Charlie]

This problem arises when the containing page has a style on the label element of float: left (or right);

I've attached a picture for clarity from one of the samples (Obviously I added the float on the label). I think we want to provide a standard experience in CKEditor regardless of CSS on the containing page (at least by default!), so I propose adding an entry:

.cke_dialog label {

float: none;

}

to each of the skins dialog.css files

Thoughts?

[Charlie]

illustration of problem

[Charlie]

Tested in nightly as well.

[Frederico Caldeira Knabben]

Replying to comp615:

The "version" field is there to tell us the oldest version that presents this problem.

[Frederico Caldeira Knabben]

Replying to comp615:

I think we want to provide a standard experience in CKEditor regardless of CSS on the containing page (at least by default!)

Your totally right... but this is a very difficult thing.

We use the reset.css for this. What we do is a deal, trying to reset the most commonly used CSS rules that could be used. But of course, CSS is so extensive that it's hard to deal with everything.

So, what should be our position here? Keep enlarging reset.css by user request? Providing instructions on how to workaround this situation (the above rule, which you have included into dialog.css, should go into your own CSS file at this point).

[Frederico Caldeira Knabben]

Replying to fredck:

Keep enlarging reset.css by user request?

We have so few requests for this (so far), that I think that this would be ok.

In this specific case... would float:none on reset.css cause problems?

[Charlie]

I don't think so? It should have the same effect as putting it in the other place. Additionally that is the default behavior (with no CSS rules right?). As I kind of alluded to (and I think you corroborated) the idea is to provide CKEditor the base CSS to start building up from, so I think your solution is perfect.

And of course, it is very hard to deal with everything, but in the end I think that is the goal is to reset as much as possible so that CKEditor works on virtually any page you could throw at it.

[Frederico Caldeira Knabben]

Having a huge reset.css file doesn't make sense either. So, let's go ahead by enlarging it when necessary.

[Frederico Caldeira Knabben]

I've checked the standard html.css file used by Firefox. Resetting float:none should not bring any issue for our UI.

[Frederico Caldeira Knabben]

Fixed with [7185].

[FrankSL]

I'm resetting this bug because it happens to me in 4.0.1, I just added a <style> tag in a simple page with framed editor like this:

<style type="text/css">
    label,select,.ui-select-menu { float: left; margin-right: 10px; font-size:1.3em; height:26px; line-height:26px; font-size:1em; }
</style>

it makes all the labels in ckeditor dialogs to be floated. Thanks

[Jakub Ś]

@FrankSL I have reported #10175 for CKE 4