@@ -188,6 +188,7 @@ module Merb::Helpers::SexyForm content else container = container || {} + add_css_class(container, "container") container.merge!(:id => "#{attrs[:id]}_container") if attrs[:id] and container[:id].blank? container_tag = Merb::Plugins.config[:merb_sexy_forms][:container_tag] tag(container_tag, content, container) @@ -200,7 +201,7 @@ module Merb::Helpers::SexyForm content else wrapper ||= {} - add_css_class(wrapper, "field") + add_css_class(wrapper, "field wrapper") tag(:div, content, wrapper) end end lib/merb-sexy-forms/builder.rb @@ -89,9 +89,14 @@ describe "sexy_form_for" do ret.should have_selector("form label.main") end - it "should add field class to field div" do + it "should add field and wrapper classes to field div" do ret = @c.render(:basic) - ret.should have_selector("form div.field") + ret.should have_selector("form div.field.wrapper") + end + + it "should add container class to container" do + ret = @c.render(:basic) + ret.should have_selector("form div.container#fake_model_foo_container") end end spec/sexy_form_for_spec.rb b4ed75b8ea9067ba6dd2d1f75fbe2728b7a09ccc Peter Sarnacki drogus@gmail.com http://github.com/drogus/merb-sexy-forms/commit/650b9eed6b91abf1ad71b5b164422b94c2cee842 650b9eed6b91abf1ad71b5b164422b94c2cee842 2009-01-20T06:50:34-08:00 2009-01-20T06:50:34-08:00 Add classes for containers and wrappers e70683c50238a0fc0f8993158911a368a50c5ad1 Peter Sarnacki drogus@gmail.com