diff options
author | Robert Speicher <rspeicher@gmail.com> | 2016-04-05 04:25:38 +0300 |
---|---|---|
committer | Robert Speicher <rspeicher@gmail.com> | 2016-04-05 04:37:07 +0300 |
commit | 7a2370f74060b2f065e3602700fe1b33fda4685c (patch) | |
tree | 0c23a588515e857598d5e389cd55aa816d3cfe35 /app/views/abuse_reports | |
parent | 2ed6cd9e469ffcdb60f21d1738de1eff8c258432 (diff) |
Standardize the way we check for and display form errors
- Some views had a "Close" button. We've removed this, because we don't
want users accidentally hiding the validation errors and not knowing
what needs to be fixed.
- Some views used `li`, some used `p`, some used `span`. We've
standardized on `li`.
- Some views only showed the first error. We've standardized on showing
all of them.
- Some views added an `#error_explanation` div, which we've made
standard.
Diffstat (limited to 'app/views/abuse_reports')
-rw-r--r-- | app/views/abuse_reports/new.html.haml | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/app/views/abuse_reports/new.html.haml b/app/views/abuse_reports/new.html.haml index 3bc1b24b5e2..06be1a53318 100644 --- a/app/views/abuse_reports/new.html.haml +++ b/app/views/abuse_reports/new.html.haml @@ -3,11 +3,9 @@ %p Please use this form to report users who create spam issues, comments or behave inappropriately. %hr = form_for @abuse_report, html: { class: 'form-horizontal js-quick-submit js-requires-input'} do |f| + = form_errors(@abuse_report) + = f.hidden_field :user_id - - if @abuse_report.errors.any? - .alert.alert-danger - - @abuse_report.errors.full_messages.each do |msg| - %p= msg .form-group = f.label :user_id, class: 'control-label' .col-sm-10 |