Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Passing block to label helper in rails3

I want to create label tag with some nested elements. I am using label helper and trying to pass inner html as block but generated HTML doesn't look as I expected. ERB:

<span>Span element</span>
<%= label("object", "method") do %>
  <span>Inner span</span>
<% end %>

HTML output:

<span>Span element</span> 
<span>Inner span</span> 

<label for="object_method">
<span>Span element</span> 
  <span>Inner span</span> 
</label>

When I pass inner html using <% %> markups output is as it should be:
ERB:

<span>Span element</span>
<%= label("object", "method") do %>
  <% raw '<span>Inner span</span>' %>
<% end %>

HTML output:

<span>Span element</span>
<label for="object_method">
  <span>Inner span</span>
</label>

I am wondering if it is my mistake or bug in ActionView label helper. For other helpers block passing works fine.

Thanks, Michał

like image 650
mzaj Avatar asked May 22 '11 13:05

mzaj


3 Answers

My understanding is that you need to use the label_tag helper in this case:

<%= label_tag "my_label_name" do %>
  <span>Inner span</span>
<% end %>

The reason for this is that although the form label helper fills out the "for" attribute for you (using your model object attribute), you don't need it with nested elements.

When you have an open label tag (rather than self-closing), that wraps the inner content, the "for" attribute is not needed, because the label is obviously associated with its nested content (this is known as implicit association).

So, this is expected behaviour - it looks like the Rails team have deliberately built it this way.

like image 89
Scott Avatar answered Nov 19 '22 21:11

Scott


Scott Lowe's answer is correct, although I'd take it one step further... You don't even need to use the Rails label_tag for this. Just use raw html like so:

<label>
  <span>Inner span</span>
</label>

If you are associating the label with a form element (like a radio button):

<label>
  <%= f.radio_button :approval_state, 'R' %>
  Rejected
</label>
like image 22
Jeff Perrin Avatar answered Nov 19 '22 22:11

Jeff Perrin


In Rails 3.2.11 this is working for me:

<span>Span element</span>
<%= label :item, :method do %>
  <span>Inner span</span>
<% end %>

Result:

<span>Span element</span>
<label for="item_method">
  <span>Inner span</span>
</label>
like image 1
fguillen Avatar answered Nov 19 '22 21:11

fguillen