(1) Take care the "cancel" of import redirects to the WP list,

(2) A successful import redirects to the import summary page and not directly back to the list,
(3) Shorten button labels

modules/bcf/app/controllers/bcf/issues_controller.rb

@@ -81,7 +81,6 @@ module ::Bcf
             @issues[:successful] << issue
           end
         end
-        redirect_to project_work_packages_bcf_issues_path(@project)
       rescue StandardError => e
         flash[:error] = I18n.t('bcf.bcf_xml.import_failed', error: e.message)
       end
@@ -98,7 +97,7 @@ module ::Bcf
             unknown_mails_action
             non_members_action].map { |key| params.dig(:import_options, key) }.include? 'cancel'
         flash[:notice] = I18n.t('bcf.bcf_xml.import_canceled')
-        redirect_to action: :index
+        redirect_to project_work_packages_bcf_issues_path(@project)
       end
     end
 

modules/bcf/app/views/bcf/issues/configure_invalid_people.html.erb

@@ -27,7 +27,7 @@
 
   <%= submit_tag t('bcf.bcf_xml.import.button_proceed'), class: 'button -highlight' %>
   <%= link_to t(:button_cancel),
-              { action: :index },
+              project_work_packages_bcf_issues_path(@project),
               class: 'button' %>
 <% end %>
 

modules/bcf/app/views/bcf/issues/configure_non_members.html.erb

@@ -42,14 +42,14 @@
 
     <%= submit_tag t('bcf.bcf_xml.import.button_proceed'), class: 'button -highlight' %>
     <%= link_to t(:button_cancel),
-                { action: :index },
+                project_work_packages_bcf_issues_path(@project),
                 class: 'button' %>
   <% else %>
     <p>
       <%= I18n.t('text_no_roles_defined') %>
     </p>
     <%= link_to t(:button_cancel),
-                { action: :index },
+                project_work_packages_bcf_issues_path(@project),
                 class: 'button' %>
   <% end %>
 <% end %>

modules/bcf/app/views/bcf/issues/configure_unknown_mails.html.erb

@@ -49,14 +49,14 @@
 
     <%= submit_tag t('bcf.bcf_xml.import.button_proceed'), class: 'button -highlight' %>
     <%= link_to t(:button_cancel),
-                { action: :index },
+                project_work_packages_bcf_issues_path(@project),
                 class: 'button' %>
   <% else %>
     <p>
       <%= I18n.t('text_no_roles_defined') %>
     </p>
     <%= link_to t(:button_cancel),
-                { action: :index },
+                project_work_packages_bcf_issues_path(@project),
                 class: 'button' %>
   <% end %>
 <% end %>

modules/bcf/app/views/bcf/issues/configure_unknown_priorities.html.erb

@@ -32,7 +32,7 @@
 
   <%= submit_tag t('bcf.bcf_xml.import.button_proceed'), class: 'button -highlight' %>
   <%= link_to t(:button_cancel),
-              { action: :index },
+              project_work_packages_bcf_issues_path(@project),
               class: 'button' %>
 <% end %>
 

modules/bcf/app/views/bcf/issues/configure_unknown_statuses.html.erb

@@ -32,7 +32,7 @@
 
   <%= submit_tag t('bcf.bcf_xml.import.button_proceed'), class: 'button -highlight' %>
   <%= link_to t(:button_cancel),
-              { action: :index },
+              project_work_packages_bcf_issues_path(@project),
               class: 'button' %>
 <% end %>
 

modules/bcf/app/views/bcf/issues/configure_unknown_types.html.erb

@@ -32,7 +32,7 @@
 
   <%= submit_tag t('bcf.bcf_xml.import.button_proceed'), class: 'button -highlight' %>
   <%= link_to t(:button_cancel),
-              { action: :index },
+              project_work_packages_bcf_issues_path(@project),
               class: 'button' %>
 <% end %>
 

modules/bcf/app/views/bcf/issues/diff_on_work_packages.html.erb

@@ -65,7 +65,7 @@
   <p><%= t('bcf.bcf_xml.import.perform_description') %></p>
   <%= submit_tag t('bcf.bcf_xml.import.button_perform_import'), class: 'button -highlight' %>
   <%= link_to t(:button_cancel),
-              { action: :index },
+              project_work_packages_bcf_issues_path(@project),
               class: 'button' %>
 <% end %>
 

modules/bcf/app/views/bcf/issues/perform_import.html.erb

@@ -1,13 +1,21 @@
 <%= stylesheet_link_tag 'bcf/bcf.css' %>
 <link rel="stylesheet" href="/highlighting/styles.css" />
 
-<h3>Failed imports of BCF topics</h3>
-<%= render partial: 'render_issues', locals: { issues: @issues[:failed] } %>
-<p></p>
-<h3>Successfully imported BCF topics</h3>
-<%= render partial: 'render_issues', locals: { issues: @issues[:successful] } %>
-<p></p>
-<hr class="form--separator" />
+<%= toolbar title: l(:label_summary) %>
+
 <%= link_to t('bcf.bcf_xml.import.button_back_to_list'),
             project_work_packages_bcf_issues_path(@project),
             class: 'button -highlight' %>
+
+<fieldset class="form--fieldset">
+  <legend class="form--fieldset-legend"><%= I18n.t('bcf.label_imported_successfully') %></legend>
+
+  <%= render partial: 'render_issues', locals: { issues: @issues[:successful] } %>
+</fieldset>
+
+
+<fieldset class="form--fieldset">
+  <legend class="form--fieldset-legend"><%= I18n.t('bcf.label_imported_failed') %></legend>
+  <%= render partial: 'render_issues', locals: { issues: @issues[:failed] } %>
+</fieldset>
+

modules/bcf/app/views/bcf/issues/upload.html.erb

@@ -14,6 +14,6 @@
 
   <%= submit_tag t('bcf.bcf_xml.import.button_prepare'), class: 'button -highlight' %>
   <%= link_to t(:button_cancel),
-              { action: :index },
+              project_work_packages_bcf_issues_path(@project),
               class: 'button' %>
 <% end %>

modules/bcf/config/locales/en.yml

@@ -2,6 +2,8 @@
 en:
   bcf:
     label_bcf: 'BCF'
+    label_imported_failed: 'Failed imports of BCF topics'
+    label_imported_successfully: 'Successfully imported BCF topics'
     issues: "Issues"
     experimental_badge: "Experimental"
     exceptions:
@@ -14,8 +16,8 @@ en:
 
     bcf_xml:
       xml_file: 'BCF XML File'
-      import_title: 'Import from BCF file'
-      export: 'Export all to BCF-XML'
+      import_title: 'Import'
+      export: 'Export'
       import_update_comment: '(Updated in BCF import)'
       import_failed: 'Cannot import BCF file: %{error}'
       import_successful: 'Imported %{count} BCF issues'

modules/bcf/config/locales/js-en.yml

@@ -3,7 +3,7 @@ en:
   js:
     bcf:
       import: 'Import'
-      export: 'Export BCF-XML'
+      export: 'Export'
       work_packages:
         default_queries:
           bcf: 'BCF issues'