Rework the new report form text and layout
authorAndy Allan <git@gravitystorm.co.uk>
Wed, 28 Feb 2018 07:27:01 +0000 (15:27 +0800)
committerAndy Allan <git@gravitystorm.co.uk>
Wed, 28 Feb 2018 07:27:01 +0000 (15:27 +0800)
app/assets/stylesheets/common.scss
app/views/reports/new.html.erb
config/locales/en.yml

index cb55f1a..f1fdc13 100644 (file)
@@ -2842,9 +2842,19 @@ input.richtext_title[type="text"] {
   float:right;
 }
 
-.disclaimer {
-  width: 600px;
+.report-disclaimer {
   background: #fff1f0;
   color: #d85030;
   border-color: rgba(216, 80, 48, 0.3);
+  padding: 5px;
+  margin-bottom: $lineheight;
+
+  ul {
+    padding-left: $lineheight;
+    margin-bottom: 0;
+
+    li {
+      list-style: disc;
+    }
+  }
 }
index f3bda88..9812c68 100644 (file)
@@ -2,13 +2,13 @@
   <h1><%= t ".title_html", :link => link_to(reportable_title(@report.issue.reportable), reportable_url(@report.issue.reportable)) %></h1>
 <% end %>
 
-<div class="disclaimer">
+<div class="report-disclaimer">
+  <%= t('issues.new.disclaimer.intro') %>
   <ul>
-    <%= t('issues.new.disclaimer.intro') %>:
-      <li> <%= t('issues.new.disclaimer.not_just_mistake') %> </li>
-      <li> <%= t('issues.new.disclaimer.unable_to_fix') %> </li>
-      <li> <%= t('issues.new.disclaimer.resolve_with_user') %> </li>
-    </ul>
+    <li> <%= t('issues.new.disclaimer.not_just_mistake') %> </li>
+    <li> <%= t('issues.new.disclaimer.unable_to_fix') %> </li>
+    <li> <%= t('issues.new.disclaimer.resolve_with_user') %> </li>
+  </ul>
 </div>
 
 <%= form_for(@report) do |f| %>
       <%= issue_form.hidden_field :reportable_type %>
     <% end %>
 
-    <div class='form-row' style='width:600px'>
-      <p><%= t('issues.new.select') %>:</p>
-
-      <div class="new-report-form">
-        <% Report.categories_for(@report.issue.reportable).each do |c| %>
-          <div style="padding-left:5px">
-            <%= radio_button :report, :category, c %>
-            <%= label_tag "report_category_#{c}", t("reports.categories.#{@report.issue.reportable.class.name}.#{c}") %> <br/>
-          </div>
-        <% end %>
-      </div>
+    <div class='form-row'>
+      <p><%= t('issues.new.select') %></p>
+      <ul>
+      <% Report.categories_for(@report.issue.reportable).each do |c| %>
+        <li>
+          <%= radio_button :report, :category, c %>
+          <%= label_tag "report_category_#{c}", t("reports.categories.#{@report.issue.reportable.class.name}.#{c}") %> <br/>
+        </li>
+      <% end %>
+      </ul>
+    </div>
 
-      <br/>
-      <%= text_area :report, :details, :cols => 20, :rows => 3, placeholder: t('issues.new.details'), required: true %>
+    <div class='form-row'>
+      <%= text_area :report, :details, :cols => 20, :rows => 5, placeholder: t('issues.new.details'), required: true %>
     </div>
 
     <div class='buttons'>
index bb1292d..40237b6 100644 (file)
@@ -941,13 +941,13 @@ en:
       successful_update: Your report has been updated successfully
       provide_details: Please provide the required details
     new:
-      details: Please provide some more details into the problem. (This field cannot be left blank!)
-      select: Select a reason for your report
+      details: Please provide some more details about the problem (required).
+      select: "Select a reason for your report:"
       disclaimer:
-        intro: Before sending in a report for official action, be sure that
-        not_just_mistake: You are sure that the problem is not just a mistake
-        unable_to_fix: You are unable to fix the problem yourself
-        resolve_with_user: You have tried to resolve the problem with the user
+        intro: "Before sending your report to the site moderators, please ensure that:"
+        not_just_mistake: You are certain that the problem is not just a mistake
+        unable_to_fix: You are unable to fix the problem yourself or with the help of your fellow community members
+        resolve_with_user: You have already tried to resolve the problem with the user concerned
     show:
       title: "%{status} Issue #%{issue_id}"
       reports: