Merge pull request #1202 from alphagov/fix-file-remove-component

Fix file remove component

Author: jamie-o-wilkinson

app/components/question/file_remove_component/view.html.erb

@@ -1,22 +1,19 @@
-<h1 class="govuk-heading-l"><%= t('forms.remove_file.show.title') %></h1>
-
-<p class="govuk-inset-text">
-  <%= question.original_filename %>
-</p>
 
 <%= form_with model: @remove_input, url: @remove_file_url, method: :delete do |form| %>
-  <div class="govuk-grid-row">
-    <div class="govuk-grid-column-two-thirds">
-      <% if @remove_input&.errors.any? %>
-        <%= form.govuk_error_summary %>
-      <% end %>
+  <% if @remove_input&.errors.any? %>
+    <%= form.govuk_error_summary %>
+  <% end %>
+
+  <h1 class="govuk-heading-l"><%= t('forms.remove_file.show.title') %></h1>
+
+  <p class="govuk-inset-text">
+    <%= question.original_filename %>
+  </p>
 
-      <%= form.govuk_collection_radio_buttons :remove,
-            @remove_input.values, ->(option) { option }, ->(option) { t('helpers.label.remove_input.options.' + "#{option}") },
-            legend: { text: t('forms.remove_file.show.radios_legend')},
-            inline: true  %>
+  <%= form.govuk_collection_radio_buttons :remove,
+        @remove_input.values, ->(option) { option }, ->(option) { t('helpers.label.remove_input.options.' + "#{option}") },
+        legend: { text: t('forms.remove_file.show.radios_legend')},
+        inline: true  %>
 
-      <%= form.govuk_submit  %>
-    </div>
-  </div>
+  <%= form.govuk_submit  %>
 <% end %>

app/components/question/file_review_component/_index.scss

@@ -0,0 +1,5 @@
+.review-file-summary-list {
+  .govuk-summary-list__key {
+    width: 20%;
+  }
+}

app/components/question/file_review_component/view.html.erb

@@ -6,7 +6,7 @@
   <%= t('forms.review_file.show.check_file') %>
 </h2>
 
-<%= govuk_summary_list do |summary_list|
+<%= govuk_summary_list(html_attributes: { class: 'review-file-summary-list' }) do |summary_list|
   summary_list.with_row do |row|
     row.with_key { t('forms.review_file.show.your_file') }
     row.with_value { question.original_filename }

app/frontend/entrypoints/application.scss

@@ -4,4 +4,5 @@ $govuk-global-styles: true;
 @import "pkg:govuk-frontend";
 @import "pkg:dfe-autocomplete";
 @import "../../components/form_header_component/";
+@import "../../components/question/file_review_component/";
 @import "../styles/app-panel";