Fieldset legend class configuration

[pduersteler]

I'm not sure if I just could not find it or if it is indeed something that I'd consider to be a missing thing, so I though I just put it up here and see what happens.

Right now, the formtastic configurations and custom input class possibilities allow for customizing almost anything, which is great. What I am currently looking for is an option to define a default class for the fieldset/legend that is being generated.

<%= f.inputs "My fieldset title" do %>
  ...
<% end %>

The code above will render the following:

<fieldset>
  <legend>
    <span>My fieldset title</span>
  </legend>
  ...
</fieldset>

As my SCSS is BEM-oriented, a configurable option to give it a default class would be very nice. There are already options in the initializer for various things like Formtastic::Helpers::FormHelper.default_form_class. Maybe something along these lines for a default legend class would be an option? I'm not sure if this would collide with the fieldset that is generated for e.g. a checkbox collection, though.

If there is already a way to modify this part, I'd happy to get a hint.

[mikz]

I know it is a poor substitute, but could you create your own semantic form builder and override:

formtastic/lib/formtastic/helpers/fieldset_wrapper.rb

Lines 51 to 57 in 60fb31e

	def field_set_legend(html_options)
	legend = (html_options[:name] || '').to_s
	# only applying if String includes '%i' avoids argument error when $DEBUG is true
	legend %= parent_child_index(html_options[:parent]) if html_options[:parent] && legend.include?('%i')
	legend = template.content_tag(:legend, template.content_tag(:span, legend.html_safe)) unless legend.blank?
	legend
	end

And the same applies for fieldset/ol

formtastic/lib/formtastic/helpers/fieldset_wrapper.rb

Lines 24 to 49 in 60fb31e

	def field_set_and_list_wrapping(*args, &block) # @private
	contents = args[-1].is_a?(::Hash) ? '' : args.pop.flatten
	html_options = args.extract_options!
	if block_given?
	contents = if template.respond_to?(:is_haml?) && template.is_haml?
	template.capture_haml(&block)
	else
	template.capture(&block)
	end
	end
	# Work-around for empty contents block
	contents ||= ""
	# Ruby 1.9: String#to_s behavior changed, need to make an explicit join.
	contents = contents.join if contents.respond_to?(:join)
	legend = field_set_legend(html_options)
	fieldset = template.content_tag(:fieldset,
	legend.html_safe << template.content_tag(:ol, contents.html_safe),
	html_options.except(:builder, :parent, :name)
	)
	fieldset
	end

Maybe give it a try in your app and see if that fixes the problem for you.

Possibly the solution could be to make it easier to modify these methods and make them a bit cleaner.

[pduersteler]

Thanks, this works so far!