Skip to content

Commit 2937ba2

Browse files
authored
Merge pull request #1131 from kennethrioja/patch-2
`flash-container` duplicate in workflows.html.erb (small fix)
2 parents 2ad373a + fd86d09 commit 2937ba2

3 files changed

Lines changed: 4 additions & 11 deletions

File tree

app/controllers/workflows_controller.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ def index
2424
# GET /workflows/1.json
2525
def show
2626
authorize @workflow
27-
@skip_flash_messages_in_header = true # we will handle flash messages in the 'workflows' layout
2827
respond_to do |format|
2928
format.html { render layout: 'workflows' }
3029
format.json

app/views/layouts/_header.html.erb

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -93,11 +93,9 @@
9393
</div>
9494
<% end %>
9595

96-
<% if !defined? @skip_flash_messages_in_header or !@skip_flash_messages_in_header %>
97-
<div id="flash-container" class="container-fluid">
98-
<%= flash_messages %>
99-
</div>
100-
<% end %>
96+
<div id="flash-container" class="container-fluid">
97+
<%= flash_messages %>
98+
</div>
10199
</header>
102100

103101
<%= render(partial: 'layouts/cookie_banner') if cookie_consent.show_banner? %>

app/views/layouts/workflows.html.erb

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,10 @@
33

44
<%= render 'layouts/head' %>
55

6-
<body>
6+
<body class=<%= 'sticky-navbar-enabled' if TeSS::Config.feature['sticky_navbar'] %> <%= 'header-notice-present' if TeSS::Config.header_notice&.strip.present? %>>
77

88
<%= render :partial=>"layouts/header" %>
99

10-
<div id="flash-container" class="workflows-container" style="margin-top: 5px">
11-
<%= flash_messages %>
12-
</div>
13-
1410
<div id="main-container" class="workflows-container">
1511
<%= render 'common/breadcrumbs' if @breadcrumbs %>
1612

0 commit comments

Comments
 (0)