Opened 16 years ago
Closed 15 years ago
#5775 closed New Feature (fixed)
Required field's label to contain a CSS class to allow it to be styled differently
| Reported by: | Joe Kavanagh | Owned by: | Tobiasz Cudnik |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.4 |
| Component: | UI : Dialogs | Version: | |
| Keywords: | IBM Confirmed | Cc: | Damian |
Description (last modified by )
If a labeled dialog UI Element's required property is set to true the field's label should contain a CSS style so it can be styled differently. For example:
<label class="cke_dialog_ui_labeled_label cke_required"...
A style like the following could then be used to mark required fields:
.cke_skin_<name> .cke_dialog_ui_labeled_label.cke_required:before {
content: "* ";
}
Attachments (2)
Change History (10)
comment:1 Changed 16 years ago by
| Keywords: | Confirmed added |
|---|
comment:2 Changed 15 years ago by
| Milestone: | CKEditor 3.x → CKEditor 3.4 |
|---|
comment:3 Changed 15 years ago by
| Description: | modified (diff) |
|---|---|
| Owner: | set to Tobiasz Cudnik |
| Status: | new → assigned |
Changed 15 years ago by
| Attachment: | 5775.patch added |
|---|
comment:4 Changed 15 years ago by
| Keywords: | Review? added |
|---|
comment:5 Changed 15 years ago by
| Keywords: | Review- added; Review? removed |
|---|
The implementation is good. Even the default styling for the required fields is a nice addition. I just have to disagree with the "*" hardcoded on the fields. We need to leave it as a CSS only solution.
By default, let's not have any "*" there, only bold, leaving to the implementer the decision to apply or not other styles to that.
Changed 15 years ago by
| Attachment: | 5775_2.patch added |
|---|
comment:6 Changed 15 years ago by
| Keywords: | Review? added; Review- removed |
|---|
Second patch doesn't contain hardcoded "*" sign.
comment:7 Changed 15 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
comment:8 Changed 15 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [5722].
Note: See
TracTickets for help on using
tickets.
We're not allowed to use such advanced CSS selectors as in the example (because of IE6's lame support for selectors), but it should be doable somehow.