Defect #5150
closed
Added by Eric Thomas about 14 years ago.
Updated about 14 years ago.
Description
In app/helpers/issues_helper.rb
def render_descendants_tree(issue)
s = '<form><table class="list issues">'
ancestors = []
issue.descendants.sort_by(&:lft).each do |child|
level = child.level - issue.level - 1
Shouldn't the table class be just "issues". I'm pretty sure CSS class names can't have spaces in them.
Also, this seems to make sense because I believe it's using this part of the css.
public/stylesheets/application.css
div.issue div.subject h3 {margin: 0; margin-bottom: 0.1em;}
#issue_tree table.issues { border: 0; }
#issue_tree td.checkbox {display:none;}
fieldset.collapsible { border-width: 1px 0 0 0; font-size: 0.9em; }
Eric Thomas wrote:
Shouldn't the table class be just "issues". I'm pretty sure CSS class names can't have spaces in them.
Nope - that's absolutely fine. The CSS class doesn't have a space in it, it is two different CSS classes to apply.
Nick Read wrote:
Eric Thomas wrote:
Shouldn't the table class be just "issues". I'm pretty sure CSS class names can't have spaces in them.
Nope - that's absolutely fine. The CSS class doesn't have a space in it, it is two different CSS classes to apply.
Ahh I see. Well is there any reason to have the "list" class since the "issues" class just takes away the border attributes that the "list" class defines?
- Status changed from New to Closed
- Resolution set to Invalid
Eric Thomas wrote:
Well is there any reason to have the "list" class since the "issues" class just takes away the border attributes that the "list" class defines?
Yes, because that is only the behavior of the default theme, other themes might do it another way. The CSS classes are to be understood as "markup" that roughly tells you what is in there, so that you can make the presentation according to that.
Okay, I guess I've got something wrong on my end. Thank you for your responses.
Also available in: Atom
PDF
Updated by 
Updated by Anonymous 
Updated by