Update template form for improved accessibility.

OregonDigital · Aug 14, 2015 · 8f3c5ba · 8f3c5ba
1 parent af3b28f
commit 8f3c5ba
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 13 deletions.
diff --git a/app/views/admin/form_templates/_form.html.erb b/app/views/admin/form_templates/_form.html.erb
@@ -3,18 +3,12 @@
   <%= f.input :title %>
 
   <%= f.simple_fields_for :properties do |p| %>
-    <div id="<%= p.object.name %>" class="panel panel-default template-property">
-      <div class="panel-heading">
-        <div class="panel-title">
-          <%= p.object.name %>
-        </div>
-      </div>
-      <div class="panel-body">
-        <%= p.input :visible, :label => "Show" %>
-        <%= p.input :required %>
-        <%= p.input :name, :as => :hidden %>
-      </div>
-    </div>
+    <fieldset id="<%= p.object.name.underscore %>" class="template-property">
+      <legend><%= p.object.name %></legend>
+      <%= p.input :visible, :label => "Show" %>
+      <%= p.input :required %>
+      <%= p.input :name, :as => :hidden %>
+    </fieldset>
   <% end %>
 
   <%= f.submit "Save Template" %>

diff --git a/spec/features/template_management_spec.rb b/spec/features/template_management_spec.rb
@@ -106,7 +106,7 @@ def navigate_to_template_admin_path
 def checkbox_states
   checkboxes = HashWithIndifferentAccess.new
   all('.template-property').each do |el|
-    checkboxes[el.find(".panel-title").text] = el.find(".form_template_properties_visible input[type=checkbox]")
+    checkboxes[el.find("legend").text] = el.find(".form_template_properties_visible input[type=checkbox]")
   end
 
   checkboxes