Merge pull request #1418 from ajb/specify-boolean-label-class-for-individual-input

specify :boolean_label_class on a per-input basis
This commit is contained in:
Lucas Mazza 2016-11-16 10:03:37 -02:00 committed by GitHub
commit 1e92d6917f
2 changed files with 16 additions and 2 deletions

View File

@ -6,7 +6,7 @@ module SimpleForm
if nested_boolean_style?
build_hidden_field_for_checkbox +
template.label_tag(nil, class: SimpleForm.boolean_label_class) {
template.label_tag(nil, class: boolean_label_class) {
build_check_box_without_hidden_field(merged_input_options) +
inline_label
}
@ -21,7 +21,7 @@ module SimpleForm
elsif nested_boolean_style?
html_options = label_html_options.dup
html_options[:class] ||= []
html_options[:class].push(SimpleForm.boolean_label_class) if SimpleForm.boolean_label_class
html_options[:class].push(boolean_label_class) if boolean_label_class
merged_input_options = merge_wrapper_options(input_html_options, wrapper_options)
@ -36,6 +36,10 @@ module SimpleForm
private
def boolean_label_class
options[:boolean_label_class] || SimpleForm.boolean_label_class
end
# Build a checkbox tag using default unchecked value. This allows us to
# reuse the method for nested boolean style, but with no unchecked value,
# which won't generate the hidden checkbox. This is the default functionality

View File

@ -164,6 +164,16 @@ class BooleanInputTest < ActionView::TestCase
end
end
test 'input boolean allows specifying boolean_label_class on a per-input basis' do
swap_wrapper do
swap SimpleForm, boolean_style: :nested, boolean_label_class: 'foo' do
with_input_for @user, :active, :boolean, boolean_label_class: 'baz'
assert_select 'label.boolean + input[type=hidden] + label.baz > input.boolean'
end
end
end
test 'input boolean with nested style works using :input only in wrapper config (no label_input), adding the extra label wrapper with custom class' do
swap_wrapper do
swap SimpleForm, boolean_style: :nested, boolean_label_class: 'foo' do