[gtk/gbsneto/fix-popover-dark-theme: 1/2] Adwaita: Use $menu_color background in popover.menu



commit fe65da05beed4fae4556185c1bfe5b15de80c29d
Author: Georges Basile Stavracas Neto <georges stavracas gmail com>
Date:   Sun Jun 30 19:47:32 2019 -0300

    Adwaita: Use $menu_color background in popover.menu
    
    GtkPopoverMenus should mimic menus. Commit d936967b7a8 introduced
    some CSS related to menu popovers, however, it hardcodes 'white'
    as the background color. That is problematic for the dark theme.
    
    Use '$menu_color' instead of 'white', since $menu_color both
    guarantees menu popovers and menus match, and already handles
    different colors for dark and light theme variants.

 gtk/theme/Adwaita/_common.scss           | 2 +-
 gtk/theme/Adwaita/gtk-contained-dark.css | 2 +-
 gtk/theme/Adwaita/gtk-contained.css      | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)
---
diff --git a/gtk/theme/Adwaita/_common.scss b/gtk/theme/Adwaita/_common.scss
index 2e257ea625..534592abb7 100644
--- a/gtk/theme/Adwaita/_common.scss
+++ b/gtk/theme/Adwaita/_common.scss
@@ -4767,7 +4767,7 @@ popover.menu {
   }
 
   &.background contents {
-    background: white;
+    background-color: $menu_color;
   }
 
   &.background separator {
diff --git a/gtk/theme/Adwaita/gtk-contained-dark.css b/gtk/theme/Adwaita/gtk-contained-dark.css
index dbc07bb542..20c0745d98 100644
--- a/gtk/theme/Adwaita/gtk-contained-dark.css
+++ b/gtk/theme/Adwaita/gtk-contained-dark.css
@@ -2063,7 +2063,7 @@ popover.menu box.inline-buttons { border-radius: 5px; border-style: none; border
 
 popover.menu box.circular-buttons { padding-bottom: 5px; }
 
-popover.menu.background contents { background: white; }
+popover.menu.background contents, popover.menu arrow { background-color: #2f2f2f; }
 
 popover.menu.background separator { margin: 5px 0px; }
 
diff --git a/gtk/theme/Adwaita/gtk-contained.css b/gtk/theme/Adwaita/gtk-contained.css
index 9b412cf499..43b1a47539 100644
--- a/gtk/theme/Adwaita/gtk-contained.css
+++ b/gtk/theme/Adwaita/gtk-contained.css
@@ -2079,7 +2079,7 @@ popover.menu box.inline-buttons { border-radius: 5px; border-style: none; border
 
 popover.menu box.circular-buttons { padding-bottom: 5px; }
 
-popover.menu.background contents { background: white; }
+popover.menu.background contents, popover.menu arrow { background-color: #ffffff; }
 
 popover.menu.background separator { margin: 5px 0px; }
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]