From 002ad820138f9737c80c70584f5e9c5835ba7892 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Oliver=20G=C3=BCnther?= <mail@oliverguenther.de>
Date: Fri, 20 Sep 2024 12:00:08 +0200
Subject: [PATCH] Revert "[57153] Reduce amount of design variables needed for
 the sidebar (#16691)"

This reverts commit ac5650ba29786145a9a85ac1ff3631f7b7fe7893.
---
 app/views/custom_styles/_inline_css.erb       | 21 ++-----------------
 .../openproject/_variable_defaults.scss       |  4 ++--
 lib/open_project/custom_styles/design.rb      |  9 +++++++-
 3 files changed, 12 insertions(+), 22 deletions(-)

diff --git a/app/views/custom_styles/_inline_css.erb b/app/views/custom_styles/_inline_css.erb
index fe6b3ddd015f..56902b4a08eb 100644
--- a/app/views/custom_styles/_inline_css.erb
+++ b/app/views/custom_styles/_inline_css.erb
@@ -31,28 +31,11 @@ See COPYRIGHT and LICENSE files for more details.
     :root {
     <% DesignColor.overwritten.each do |design_color| %>
         --<%= design_color.variable %>: <%= design_color.hexcode %>;
-    <% if design_color.variable == "header-bg-color" %>
-      --header-item-font-color: <%= design_color.contrasting_font_color %>;
-      <% if design_color.bright? %>
+    <% if design_color.variable == "header-border-bottom-color" %>
         --header-border-bottom-width: 1px;
-        --header-border-bottom-color: #d0d7de;
-      <% end %>
     <% end %>
-    <% if design_color.variable == "header-item-bg-hover-color" %>
-      --header-item-font-hover-color: <%= design_color.contrasting_font_color %>;
-    <% end %>
-    <% if design_color.variable == "main-menu-bg-color" %>
-      --main-menu-font-color: <%= design_color.contrasting_font_color %>;
-      <% if design_color.bright? %>
+    <% if design_color.variable == "main-menu-border-color" %>
         --main-menu-border-width: 1px;
-        --main-menu-border-color: #d0d7de;
-      <% end %>
-    <% end %>
-    <% if design_color.variable == "main-menu-bg-selected-background" %>
-      --main-menu-selected-font-color: <%= design_color.contrasting_font_color %>;
-    <% end %>
-    <% if design_color.variable == "main-menu-bg-hover-background" %>
-      --main-menu-hover-font-color: <%= design_color.contrasting_font_color %>;
     <% end %>
     <% if design_color.variable == "primary-button-color" %>
         --primary-button-color--major1: <%= design_color.darken 0.18 %>;
diff --git a/frontend/src/global_styles/openproject/_variable_defaults.scss b/frontend/src/global_styles/openproject/_variable_defaults.scss
index 46b564f25da3..57f165baedbe 100644
--- a/frontend/src/global_styles/openproject/_variable_defaults.scss
+++ b/frontend/src/global_styles/openproject/_variable_defaults.scss
@@ -86,9 +86,9 @@
   --main-menu-bg-selected-background: #175A8E;
   --main-menu-bg-hover-background: #124E7C;
   --main-menu-font-color: #FFFFFF;
-  --main-menu-hover-font-color: #FFFFFF;
+  --main-menu-hover-font-color: var(--main-menu-font-color);
   --main-menu-resizer-color: var(--main-menu-border-color);
-  --main-menu-selected-font-color: #FFFFFF;
+  --main-menu-selected-font-color: var(--main-menu-font-color);
   --main-menu-font-size: 14px;
   --main-menu-fieldset-header-color: #B0B2B3;
   --main-menu-hover-border-color: transparent;
diff --git a/lib/open_project/custom_styles/design.rb b/lib/open_project/custom_styles/design.rb
index 28f2dc955670..2da19415007f 100644
--- a/lib/open_project/custom_styles/design.rb
+++ b/lib/open_project/custom_styles/design.rb
@@ -72,9 +72,16 @@ def customizable_variables
           accent-color
           header-bg-color
           header-item-bg-hover-color
+          header-item-font-color
+          header-item-font-hover-color
+          header-border-bottom-color
           main-menu-bg-color
           main-menu-bg-selected-background
-          main-menu-bg-hover-background)
+          main-menu-bg-hover-background
+          main-menu-font-color
+          main-menu-selected-font-color
+          main-menu-hover-font-color
+          main-menu-border-color )
     end
   end
 end