From 2c2c5b926a0142b579f4f3a92b95703f754594e0 Mon Sep 17 00:00:00 2001 From: Alexander Date: Sun, 10 Nov 2013 12:28:38 +0200 Subject: [PATCH 1/3] Allow to override the wrapper div class name --- lib/nested_form/builder_mixin.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/nested_form/builder_mixin.rb b/lib/nested_form/builder_mixin.rb index 26a98877..e56f8e09 100644 --- a/lib/nested_form/builder_mixin.rb +++ b/lib/nested_form/builder_mixin.rb @@ -88,7 +88,7 @@ def fields_for_with_nested_attributes(association_name, *args) end def fields_for_nested_model(name, object, options, block) - classes = 'fields' + classes = options.fetch(:nested_wrapper_class, 'fields') classes << ' marked_for_destruction' if object.respond_to?(:marked_for_destruction?) && object.marked_for_destruction? perform_wrap = options.fetch(:nested_wrapper, true) From 500c3647ec13acd019c62de5b6ab57b315a2f0b6 Mon Sep 17 00:00:00 2001 From: Alexander Date: Sun, 10 Nov 2013 12:34:51 +0200 Subject: [PATCH 2/3] Ensure the "fields" class is always added to the wrapper --- lib/nested_form/builder_mixin.rb | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lib/nested_form/builder_mixin.rb b/lib/nested_form/builder_mixin.rb index e56f8e09..c887a3fd 100644 --- a/lib/nested_form/builder_mixin.rb +++ b/lib/nested_form/builder_mixin.rb @@ -88,14 +88,15 @@ def fields_for_with_nested_attributes(association_name, *args) end def fields_for_nested_model(name, object, options, block) - classes = options.fetch(:nested_wrapper_class, 'fields') + classes = options.fetch(:nested_wrapper_class, '') + classes << ' fields' classes << ' marked_for_destruction' if object.respond_to?(:marked_for_destruction?) && object.marked_for_destruction? perform_wrap = options.fetch(:nested_wrapper, true) perform_wrap &&= options[:wrapper] != false # wrap even if nil if perform_wrap - @template.content_tag(:div, super, :class => classes) + @template.content_tag(:div, super, :class => classes.strip) else super end From 405e4b2e45e907aefc071a2b2e5001e21df93d7f Mon Sep 17 00:00:00 2001 From: Alexander Date: Sat, 31 Jan 2015 18:58:45 +0200 Subject: [PATCH 3/3] Do not mutate original string --- lib/nested_form/builder_mixin.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/nested_form/builder_mixin.rb b/lib/nested_form/builder_mixin.rb index c887a3fd..83727a84 100644 --- a/lib/nested_form/builder_mixin.rb +++ b/lib/nested_form/builder_mixin.rb @@ -88,7 +88,7 @@ def fields_for_with_nested_attributes(association_name, *args) end def fields_for_nested_model(name, object, options, block) - classes = options.fetch(:nested_wrapper_class, '') + classes = options.fetch(:nested_wrapper_class, '').dup classes << ' fields' classes << ' marked_for_destruction' if object.respond_to?(:marked_for_destruction?) && object.marked_for_destruction?