Skip to content

Commit

Permalink
[BUG] Fix admin layout
Browse files Browse the repository at this point in the history
- Fix the layout of admin pages, it previously was full-width and had
the alert at the incorrect place and within an container.
- Make the placement of the alert consistent with other pages, inside
`flex-container-main` and not wrapped around a container.
- We have to revert 145bebc, as this
expected that the page contain provided padding, this was provided by
the incorrect placement of the alert. As well isn't consistent with how
other pages are being shown, non-full width. The solution to the
described problem isn't optimal and should rather be fixed with the tables.
- Reverts 145bebc
- Resolves #3082
  • Loading branch information
Gusted committed Apr 6, 2024
1 parent 0a51ae2 commit c2d0f64
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions templates/admin/layout_head.tmpl
Original file line number Diff line number Diff line change
@@ -1,11 +1,9 @@
{{template "base/head" .ctxData}}
<div role="main" aria-label="{{.ctxData.Title}}" class="page-content {{.pageClass}}">
<div class="ui container">
{{template "base/alert" .ctxData}}
</div>
<div class="ui container fluid padded flex-container">
<div class="ui container flex-container">
{{template "admin/navbar" .ctxData}}
<div class="flex-container-main">
{{template "base/alert" .ctxData}}
{{/* block: admin-setting-content */}}

{{if false}}{{/* to make html structure "likely" complete to prevent IDE warnings */}}
Expand Down

0 comments on commit c2d0f64

Please sign in to comment.