Adwaita: Fix broken osd popover style
authornana-4 <hnmaigo@gmail.com>
Mon, 2 Sep 2019 16:34:05 +0000 (01:34 +0900)
committernana-4 <hnmaigo@gmail.com>
Mon, 2 Sep 2019 16:34:05 +0000 (01:34 +0900)
Apply %osd to child arrow and contents instead of parent popover.

https://gitlab.gnome.org/GNOME/gtk/issues/2061

gtk/theme/Adwaita/_common.scss

index d5945fdca360caef2b877ed8a97d6b94e0c3b2c2..0432b2a24392b6f891d192e2dca4a06b7f693dbf 100644 (file)
@@ -2187,59 +2187,54 @@ menuitem {
 
 popover.background {
   background-color: transparent;
-  padding: 0px;
-}
-
-popover.menu>arrow,
-popover>arrow {
-  background-color: $popover_bg_color;
-  border: 1px solid $borders_color;
-}
 
-popover>contents {
-  padding: 8px;
-  background-color: $popover_bg_color;
-  border: 1px solid $borders_color;
-  margin: 0px;
-}
+  > arrow,
+  > contents {
+    background-color: $popover_bg_color;
+    border: 1px solid $borders_color;
+    box-shadow: 0 1px 2px transparentize(black, 0.7);
 
-popover.background>contents {
+    &:backdrop {
+      background-color: $backdrop_bg_color;
+      border-color: $backdrop_borders_color;
+      box-shadow: none;
+    }
+  }
 
-  background-color: $popover_bg_color;
-  box-shadow: 0 1px 2px transparentize(black, 0.7);
-  .csd &, & {
-    border: 1px solid $borders_color;
+  > contents {
+    padding: 8px;
     border-radius: $popover_radius;
-  }
 
-  &:backdrop {
-    background-color: $backdrop_bg_color;
-    box-shadow: none;
+    > list,
+    > .view,
+    > toolbar {
+      border-style: none;
+      background-color: transparent;
+    }
+
+    separator { margin: 3px; }
+
+    list separator { margin: 0; }
   }
 
-  > list,
-  > .view,
-  > toolbar {
-    border-style: none;
+  .osd &,
+  &.touch-selection,
+  &.magnifier {
     background-color: transparent;
-  }
 
-  .csd &, & {
-    &.touch-selection,
-    &.magnifier {
+    > arrow,
+    > contents {
       @extend %osd;
 
       border: 1px solid transparentize(white, 0.9);
-
-      button { @extend %osd_button };
+      box-shadow: none;
     }
-
-    &.osd { @extend %osd; }
   }
 
-  separator { margin: 3px; }
-
-  list separator { margin: 0px; }
+  &.touch-selection,
+  &.magnifier {
+    button { @extend %osd_button; }
+  }
 }