From 3965c693151854b675984a7e47f263966c70cc9f Mon Sep 17 00:00:00 2001
From: silverwind <me@silverwind.io>
Date: Fri, 21 Jul 2023 19:37:26 +0200
Subject: [PATCH] Reduce margins on user profile page

---
 templates/admin/layout_head.tmpl         |  4 ++--
 templates/admin/navbar.tmpl              |  2 +-
 templates/user/settings/layout_head.tmpl |  4 ++--
 templates/user/settings/navbar.tmpl      |  2 +-
 web_src/css/admin.css                    | 22 ----------------------
 web_src/css/index.css                    |  1 +
 web_src/css/modules/flexcontainer.css    | 23 +++++++++++++++++++++++
 7 files changed, 30 insertions(+), 28 deletions(-)
 create mode 100644 web_src/css/modules/flexcontainer.css

diff --git a/templates/admin/layout_head.tmpl b/templates/admin/layout_head.tmpl
index 71410ea36ba60..8ba47f2f14aa4 100644
--- a/templates/admin/layout_head.tmpl
+++ b/templates/admin/layout_head.tmpl
@@ -1,8 +1,8 @@
 {{template "base/head" .ctxData}}
 <div role="main" aria-label="{{.ctxData.Title}}" class="page-content {{.pageClass}}">
-	<div class="ui container admin-container">
+	<div class="ui container flex-container">
 		{{template "admin/navbar" .ctxData}}
-		<div class="admin-main">
+		<div class="flex-container-main">
 			{{template "base/alert" .ctxData}}
 			{{/* block: admin-setting-content */}}
 
diff --git a/templates/admin/navbar.tmpl b/templates/admin/navbar.tmpl
index a96e5c93677a5..070b1acfe24e7 100644
--- a/templates/admin/navbar.tmpl
+++ b/templates/admin/navbar.tmpl
@@ -1,4 +1,4 @@
-<div class="admin-nav">
+<div class="flex-container-nav">
 	<div class="ui fluid vertical menu">
 		<div class="header item">{{.locale.Tr "settings"}}</div>
 		<a class="{{if .PageIsAdminDashboard}}active {{end}}item" href="{{AppSubUrl}}/admin">
diff --git a/templates/user/settings/layout_head.tmpl b/templates/user/settings/layout_head.tmpl
index 26d1653948746..dce496e7fae1b 100644
--- a/templates/user/settings/layout_head.tmpl
+++ b/templates/user/settings/layout_head.tmpl
@@ -1,8 +1,8 @@
 {{template "base/head" .ctxData}}
 <div role="main" aria-label="{{.ctxData.Title}}" class="page-content {{.pageClass}}">
-	<div class="ui container stackable grid">
+	<div class="ui container flex-container">
 		{{template "user/settings/navbar" .ctxData}}
-		<div class="twelve wide column">
+		<div class="flex-container-main">
 			{{template "base/alert" .ctxData}}
 			{{/* block: user-setting-content */}}
 
diff --git a/templates/user/settings/navbar.tmpl b/templates/user/settings/navbar.tmpl
index 7612e41ba5a1d..1278e4a7972f6 100644
--- a/templates/user/settings/navbar.tmpl
+++ b/templates/user/settings/navbar.tmpl
@@ -1,4 +1,4 @@
-<div class="four wide column">
+<div class="flex-container-nav">
 	<div class="ui fluid vertical menu">
 		<div class="header item">{{.locale.Tr "org.settings"}}</div>
 		<a class="{{if .PageIsSettingsProfile}}active {{end}}item" href="{{AppSubUrl}}/user/settings">
diff --git a/web_src/css/admin.css b/web_src/css/admin.css
index 8c9d1e201c2a7..f4848bef4af5e 100644
--- a/web_src/css/admin.css
+++ b/web_src/css/admin.css
@@ -1,25 +1,3 @@
-.admin-container {
-  display: flex !important;
-  gap: 16px;
-}
-
-.admin-nav {
-  width: 240px;
-}
-
-.admin-main {
-  flex: 1;
-}
-
-@media (max-width: 767.98px) {
-  .admin-container {
-    flex-direction: column;
-  }
-  .admin-nav {
-    width: auto;
-  }
-}
-
 .admin.hooks .list > .item:not(:first-child) {
   border-top: 1px solid var(--color-secondary);
   padding: 0.25rem 1rem;
diff --git a/web_src/css/index.css b/web_src/css/index.css
index fa918f49dad25..230e032ac05b6 100644
--- a/web_src/css/index.css
+++ b/web_src/css/index.css
@@ -11,6 +11,7 @@
 @import "./modules/toast.css";
 @import "./modules/divider.css";
 @import "./modules/svg.css";
+@import "./modules/flexcontainer.css";
 
 @import "./shared/issuelist.css";
 @import "./shared/milestone.css";
diff --git a/web_src/css/modules/flexcontainer.css b/web_src/css/modules/flexcontainer.css
new file mode 100644
index 0000000000000..1721f14b2cafb
--- /dev/null
+++ b/web_src/css/modules/flexcontainer.css
@@ -0,0 +1,23 @@
+/* container for full-page content with sidebar on left */
+
+.flex-container {
+  display: flex !important;
+  gap: 16px;
+}
+
+.flex-container-nav {
+  width: 240px;
+}
+
+.flex-container-main {
+  flex: 1;
+}
+
+@media (max-width: 767.98px) {
+  .flex-container {
+    flex-direction: column;
+  }
+  .flex-container-nav {
+    width: auto;
+  }
+}