Closed (fixed)
Project:
Zen
Version:
6.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
13 Apr 2008 at 14:51 UTC
Updated:
28 Apr 2008 at 17:12 UTC
There's a declaration in zen.css that says:
.form-item label /* The label for a form element */
{
display: block;
font-weight: bold;
}
I'm not sure of it's purpose, but it breaks the .content-inline declaration from system.css (which looks like this):
/*
** Inline items (need to override above)
*/
.container-inline div, .container-inline label {
display: inline;
}
The declaration in zen.css takes precedence and form labels don't show up inline. I'm not sure whether it's better to just remove the display: block; from that line or to add a separate declaration after it to allow for inline stuff. Taking the "more is more" philosophy, I've created a patch that adds another declaration to zen.css.
If the purpose of the top rule can be determined, then we should probably keep it in and apply this patch. Otherwise, maybe we can just delete that rule from zen.css and call it a day.
| Comment | File | Size | Author |
|---|---|---|---|
| zen-label-inline.patch | 556 bytes | jjeff |
Comments
Comment #1
johnalbinOk, I committed to HEAD and DRUPAL-5.
BTW, I don't see any forms that get broken in Core, but no core modules have a label in their inline forms. I'm assuming this breakage in a contrib module.
Comment #2
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.