Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#7943 closed Bug (fixed)

Label Style should have float:none

Reported by: comp615 Owned by: fredck
Priority: Normal Milestone: CKEditor 3.6.2
Component: UI : Skins Version: 3.6
Keywords: Cc:

Description

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?

Attachments (2)

pic.jpg (336.7 KB) - added by comp615 5 years ago.
illustration of problem
7943.patch (1.2 KB) - added by fredck 5 years ago.

Download all attachments as: .zip

Change History (14)

Changed 5 years ago by comp615

illustration of problem

comment:1 follow-up: Changed 5 years ago by comp615

  • Version changed from 3.6 to 3.6.1 (SVN - trunk)

Tested in nightly as well.

comment:2 in reply to: ↑ 1 Changed 5 years ago by fredck

  • Component changed from UI : Dialogs to UI : Skins
  • Keywords label dialog css removed
  • Version changed from 3.6.1 (SVN - trunk) to 3.6

Replying to comp615:

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

comment:3 in reply to: ↑ description ; follow-up: Changed 5 years ago by fredck

  • Keywords Discussion added
  • Status changed from new to confirmed

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).

comment:4 in reply to: ↑ 3 Changed 5 years ago by fredck

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?

comment:5 Changed 5 years ago by comp615

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.

comment:6 Changed 5 years ago by fredck

  • Keywords Discussion removed
  • Owner set to fredck
  • Status changed from confirmed to assigned

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

Changed 5 years ago by fredck

comment:7 Changed 5 years ago by fredck

  • Status changed from assigned to review

comment:8 Changed 5 years ago by fredck

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

comment:9 Changed 5 years ago by Saare

  • Status changed from review to review_passed

comment:10 Changed 5 years ago by fredck

  • Milestone set to CKEditor 3.6.2
  • Resolution set to fixed
  • Status changed from review_passed to closed

Fixed with [7185].

comment:11 Changed 4 years ago by FrankSL

  • Version changed from 3.6 to 4.0.1

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

comment:12 Changed 4 years ago by j.swiderski

  • Version changed from 4.0.1 to 3.6

@FrankSL I have reported #10175 for CKE 4

Note: See TracTickets for help on using tickets.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy