Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
theme system
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
24 Mar 2009 at 12:07 UTC
Updated:
19 Sep 2009 at 06:10 UTC
Jump to comment: Most recent file

Comments
Comment #1
mr.baileysWhat browser are you using? When I visit the same page using a recently checked out copy, the descriptions are more or less indented as they should (see screenshot, Firefox 3 on windows)
Comment #2
catchSorry I should have mentioned that this is with theme('indentation') removed from comment_form_alter(). We shouldn't need to add nasty empty divs there to get around them looking like crap.
Comment #3
stborchertThis would only be possible if the surrounding divs get an additional class representing the item type, e.g.
If this is done it shouldn't be a problem to style the description of single element types.
Comment #4
stborchertPostponed until #43493: FAPI: Add name/type as CSS class for form elements gets in.
If this issue is fixed this one can easily be done with
Comment #5
stborchertJust noticed the issue was still open ...
Here's the (very simple) patch.
Comment #6
stborchertRe-roll with indentation of
2.4em. This fits better.Comment #7
eigentor commentedChecked: wonderful. This fits on point.
Comment #8
webchickHm. I'm not sure this is any longer valid? The descriptions of the comment settings in the vertical tabs on the node add form look fine to me under Garland. This patch, otoh, ends up wildly skewing them to the right. They are a bit off in Seven, but it strikes me that these ought to maybe be fixed there instead.
Could you take another look?
Also, because this changes no APIs, this is something that we can kick to post-code freeze.
Comment #9
stborchertAh, I see. comment.module is using
theme('indentation')to indent the description. Catch noted this before but I forgot about it.I removed this theme-calls from comment.module and now it looks good.
(btw.: there are no further calls of
theme_indentationso we can actually remove it)I don't "installed" Seven for now but this could be a follow up issue (if you like).
Comment #10
catchstBorchert, please double check theme('indentation') doesn't get called indirectly via drupal_render() - I think it's still used by drag and drop but might not be called directly any more. If it really can be removed, this issue should be marked critical instead of minor :p
Comment #11
stborchertOk,
theme_indentationmight be removed in an other issue. I forgot to update the code base so here is a new patch with fixed descriptions in "Seven".Comment #12
stborchertRe-roll (
.form-item-checkboxis now.form-type-checkbox).Comment #13
dawehnerthere is a line to much ;)
This review is powered by Dreditor.
Comment #14
stborchertUpdated patch and removed the blank line.
Thanks, dereine! :-)
Comment #15
catchLooks great.
Comment #16
dries commentedCommitted to CVS HEAD. Thanks.