Added form error styling and associated tests. Closes out codeforamer…

…ica#188.
alexi215 · Aug 7, 2014 · c3c835b · c3c835b
1 parent 914cb78
commit c3c835b
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 0 deletions.
diff --git a/app/assets/stylesheets/application.css.scss b/app/assets/stylesheets/application.css.scss
@@ -100,6 +100,17 @@
   }
 }
 
+// Adds a red border around invalid input fields after form submission.
+.field_with_errors
+{
+  input
+  {
+    border-color: red;
+    border-width: 2px;
+    margin-left: 10px;
+  }
+}
+
 .add_fields, .add_array_fields
 {
   margin: 10px 0px 15px 10px;

diff --git a/spec/features/admin/locations/create_location_spec.rb b/spec/features/admin/locations/create_location_spec.rb
@@ -27,6 +27,7 @@
     expect(page).to have_content "Description can't be blank for Location"
     expect(page).to have_content "Name can't be blank for Location"
     expect(page).to have_content 'A location must have at least one address type'
+    expect(page).to have_css('.field_with_errors')
   end
 
   scenario 'with valid mailing address', :js do

diff --git a/spec/features/admin/organizations/create_organization_spec.rb b/spec/features/admin/organizations/create_organization_spec.rb
@@ -18,6 +18,7 @@
   scenario 'without any required fields' do
     click_button 'Create organization'
     expect(page).to have_content "Name can't be blank for Organization"
+    expect(page).to have_css('.field_with_errors')
   end
 
   scenario 'when adding a website', :js do