[53809] Update PageHeaders & SubHeaders in the (rails) global index pages (fix issues)

app/views/homescreen/index.html.erb

@@ -33,11 +33,6 @@ See COPYRIGHT and LICENSE files for more details.
   end
 %>
 
-<h2 class="headline--application">
-  <span><%= organization_icon %></span>
-  <%= organization_name %>
-</h2>
-
 <%= render partial: 'announcements/show' %>
 
 <% if @homescreen[:blocks].any? %>

modules/boards/app/controllers/boards/boards_controller.rb

@@ -16,13 +16,22 @@ class BoardsController < BaseController
     menu_item :boards
 
     def index
+      @show_create_button = show_create_button
       render "index", locals: { menu_name: project_or_global_menu }
     end
 
     def show
       render layout: "angular/angular"
     end
 
+    def show_create_button
+      if @project
+        User.current.allowed_in_project?(:manage_board_views, @project)
+      else
+        User.current.allowed_in_any_project?(:manage_board_views)
+      end
+    end
+
     def default_breadcrumb; end
 
     def show_local_breadcrumb

modules/boards/app/views/boards/boards/index.html.erb

@@ -32,19 +32,28 @@ See COPYRIGHT and LICENSE files for more details.
 <%=
   render Primer::OpenProject::PageHeader.new do |header|
     header.with_title { t("boards.label_boards") }
-    header.with_breadcrumbs([{ href: home_path, text: organization_name},
+    header.with_breadcrumbs([{ href: home_path, text: organization_name },
                              *([href: project_overview_path(@project.id), text: @project.name] if @project),
                              t("boards.label_boards")])
   end
 %>
 
-<%=
-  render Primer::OpenProject::SubHeader.new do |subheader|
-    subheader.with_action_component(data: { "test-selector": "add-board-button"}) do
-       render Boards::AddButtonComponent.new(current_project: @project)
+<% if @show_create_button %>
+  <%=
+    render Primer::OpenProject::SubHeader.new do |subheader|
+      subheader.with_action_button(scheme: :primary,
+                                   aria: { label: I18n.t("boards.label_create_new_board") },
+                                   title: I18n.t("boards.label_board"),
+                                   tag: :a,
+                                   id: "add-board-button",
+                                   href: polymorphic_path([:new, @project, :work_package_board]),
+                                   data: { "test-selector": "add-board-button"}) do |button|
+        button.with_leading_visual_icon(icon: :plus)
+        I18n.t("boards.label_board")
+      end
     end
-  end
-%>
+  %>
+<% end %>
 
 <% if @board_grids.empty? -%>
   <%= no_results_box %>

modules/boards/app/views/boards/boards/overview.html.erb

@@ -29,8 +29,30 @@ See COPYRIGHT and LICENSE files for more details.
 
 <% html_title(t('boards.label_boards')) -%>
 
-<%= toolbar title: t('boards.label_boards') do %>
-  <%= render Boards::AddButtonComponent.new %>
+<%=
+  render Primer::OpenProject::PageHeader.new do |header|
+    header.with_title { t("boards.label_boards") }
+    header.with_breadcrumbs([{ href: home_path, text: organization_name},
+                             *([href: project_overview_path(@project.id), text: @project.name] if @project),
+                             t("boards.label_boards")])
+  end
+%>
+
+<% if @show_create_button %>
+  <%=
+    render Primer::OpenProject::SubHeader.new do |subheader|
+      subheader.with_action_button(scheme: :primary,
+                                   aria: { label: I18n.t("boards.label_create_new_board") },
+                                   title: I18n.t("boards.label_board"),
+                                   tag: :a,
+                                   id: "add-board-button",
+                                   href: polymorphic_path([:new, @project, :work_package_board]),
+                                   data: { "test-selector": "add-board-button"}) do |button|
+        button.with_leading_visual_icon(icon: :plus)
+        I18n.t("boards.label_board")
+      end
+    end
+  %>
 <% end %>
 
 <% if @board_grids.empty? -%>

modules/boards/spec/features/support/board_index_page.rb

@@ -59,9 +59,7 @@ def expect_board(name, present: true)
 
     def create_board(action: "Basic", title: "#{action} Board", expect_empty: false, via_toolbar: true)
       if via_toolbar
-        within '[data-test-selector="add-board-button"]' do
-          click_link "Board"
-        end
+        find('[data-test-selector="add-board-button"]').click
       else
         find('[data-test-selector="boards--create-button"]').click
       end

modules/boards/spec/features/support/board_list_page.rb

@@ -35,15 +35,11 @@ def visit!
     end
 
     def expect_create_button
-      within '[data-test-selector="add-board-button"]' do
-        expect(page).to have_link "Board"
-      end
+      expect(page).to have_css('[data-test-selector="add-board-button"]', text: "Board")
     end
 
     def expect_no_create_button
-      within '[data-test-selector="add-board-button"]' do
-        expect(page).to have_no_link "Board"
-      end
+      expect(page).to have_no_css('[data-test-selector="add-board-button"]', text: "Board")
     end
 
     def expect_delete_buttons(*boards)

modules/boards/spec/features/support/board_new_page.rb

@@ -13,9 +13,7 @@ def visit!
     def navigate_by_create_button
       visit work_package_boards_path unless page.current_path == work_package_boards_path
 
-      within '[data-test-selector="add-board-button"]' do
-        click_link "Board"
-      end
+      find('[data-test-selector="add-board-button"]').click
     end
 
     def set_title(title)

modules/calendar/app/controllers/calendar/calendars_controller.rb

@@ -42,6 +42,8 @@ class CalendarsController < ApplicationController
 
     def index
       @views = visible_views
+      @show_create_button = show_create_button?
+      @dynamic_path = dynamic_path
       render "index", locals: { menu_name: project_or_global_menu }
     end
 
@@ -109,5 +111,21 @@ def find_calendar
     rescue ActiveRecord::RecordNotFound
       render_404
     end
+
+    def show_create_button?
+      if @project
+        User.current.allowed_in_project?(:manage_calendars, @project)
+      else
+        User.current.allowed_in_any_project?(:manage_calendars)
+      end
+    end
+
+    def dynamic_path
+      if @project
+        new_project_calendars_path(@project)
+      else
+        new_calendar_path
+      end
+    end
   end
 end