Added label as a field for multiparameter_field #97

Open
wants to merge 1 commit into
from

1 participant

@ericboehs

This may need some clean up (label for doesn't have the id of the next field). I'm sure other html tags can be used (divs and what not). Not sure if you want to use a white list or a blacklist here.

Also you may want to document and test this. I'm just submitting so you have an idea of what would be use and how to implement it.

Here's how I use it:

<%= f.multiparameter_field :by_check_in_and_rate,
{ field_type: :label, name: 'Check In/Out Times' },
{ field_type: :text_field, placeholder: 'Check in' },
{ field_type: :text_field, placeholder: 'Check out' },
{ field_type: :label, name: 'Price/Night (USD)' },
{ field_type: :text_field, placeholder: 'Min $' },
{ field_type: :text_field, placeholder: 'Max $' },
size: 5 %>

@ericboehs ericboehs commented on the diff Apr 26, 2012
lib/meta_search/helpers/form_builder.rb
objectify_options(opts))
)
+ fields_count += 1 unless type == :label
@ericboehs
ericboehs added a line comment Apr 26, 2012

Oops too far indented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@ericboehs ericboehs commented on the diff Apr 26, 2012
lib/meta_search/helpers/form_builder.rb
args.each_with_index do |field, index|
type = field.delete(:field_type) || raise(ArgumentError, "No :field_type specified.")
cast = field.delete(:type_cast) || ''
opts = defaults.merge(field)
+ field_name = opts[:name] || method.to_s + "(#{fields_count + 1}#{cast})"
@ericboehs
ericboehs added a line comment Apr 26, 2012

Could probably put the fields_count line before this one and then not use +1 on this line.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment