Opened 16 years ago

Closed 16 years ago

Last modified 14 years ago

#11422 closed (duplicate)

Broken html in CheckboxSelectMultiple widget (with fix)

Reported by: Jerome Leclanche Owned by: bmihelac
Component: Forms Version: dev
Severity: Keywords:
Cc: adys.wh@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The html in CheckboxSelectMultiple widget is broken, under the format <li><label%s>%s %s</label></li>. This itches Gecko (noticeable with firebug), and completely breaks webkit.

Attached patch, and before/after screenshot.

Attachments (4)

checkboxselectmultiple.patch​ (721 bytes ) - added by Jerome Leclanche 16 years ago.
Patch
beforeafter.png​ (28.5 KB ) - added by Jerome Leclanche 16 years ago.
Before / After (Webkit)
Screen shot 2010-02-28 at 11.12.18 AM.png​ (11.1 KB ) - added by Idan Gazit 16 years ago.
Rendering in RTL locales
11422.diff​ (418 bytes ) - added by bmihelac 16 years ago.

Download all attachments as: .zip

Change History (17)

by Jerome Leclanche, 16 years ago

Attachment: checkboxselectmultiple.patch​ added

Patch

by Jerome Leclanche, 16 years ago

Attachment: beforeafter.png​ added

Before / After (Webkit)

comment:1 by Jerome Leclanche, 16 years ago

Cc: adys.wh@… added

comment:2 by Jerome Leclanche, 16 years ago

Why hasn't this been committed yet?

comment:3 by Russell Keith-Magee, 16 years ago

milestone: → 1.2
Needs tests: set
Triage Stage: Unreviewed → Accepted

comment:4 by jkocherhans, 16 years ago

This patch looks fine, but I think the checkboxes should be on the left, not the right. The css is the culprit here. Anyone want to fix it and upload a new patch?

by Idan Gazit, 16 years ago

Attachment: Screen shot 2010-02-28 at 11.12.18 AM.png​ added

Rendering in RTL locales

comment:5 by Idan Gazit, 16 years ago

Existing patch also breaks down in RTL locales, see attached screenie (webkit)

comment:6 by Russell Keith-Magee, 16 years ago

See #13131 for a proposed fix for the RTL issue.

comment:7 by Patryk Zawadzki, 16 years ago

The HTML is not broken. What is broken are CSS rules applied to the widget in the admin app. I have proposed a fix in #13131 (which was closed as a duplicate of this one).

comment:8 by bmihelac, 16 years ago

Owner: changed from Jerome Leclanche to bmihelac

I agree with patrys, CSS for admin is problematic and not HTML.
In forms.css labels are floated to the left which is not ok good for multiple checkboxes because it makes whole li element floated.
The simplest solution is to remove floating form label if it is inside li element.
I am attaching patch for this.

by bmihelac, 16 years ago

Attachment: 11422.diff​ added

comment:9 by bmihelac, 16 years ago

I just found that this issue is duplicate of #12260

comment:10 by Patryk Zawadzki, 16 years ago

This one is was reported earlier so I'd say #12260 is a duplicate of this one.

in reply to:  10 comment:11 by bmihelac, 16 years ago

Replying to patrys:

This one is was reported earlier so I'd say #12260 is a duplicate of this one.

#12260 better describes the actual problem and looking at it now, it also reset label to display and width settings.
I think it is safe to close this one as invalid.

comment:12 by stemmetje, 16 years ago

Resolution: → duplicate
Status: new → closed

This issue was fixed under ticket #12260, so I'm marking this ticket as a duplicate.

comment:13 by Jacob, 14 years ago

milestone: 1.2

Milestone 1.2 deleted

Note: See TracTickets for help on using tickets.
Back to Top