From 1263f84c3f27c4da61d6ee90619e615c47333f9e Mon Sep 17 00:00:00 2001
From: nana-4 <hnmaigo@gmail.com>
Date: Sat, 25 Jan 2020 12:15:54 +0900
Subject: [PATCH] theme: Make %notification_bubble a mixin

And simplify .datemenu-today-button styling by using it.

This allows removing duplicate code for flat notification_bubble
styling.

https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/957
---
 data/theme/gnome-shell-sass/_common.scss         | 11 +++++++----
 .../gnome-shell-sass/widgets/_calendar.scss      | 16 +++-------------
 .../gnome-shell-sass/widgets/_message-list.scss  |  2 +-
 3 files changed, 11 insertions(+), 18 deletions(-)

diff --git a/data/theme/gnome-shell-sass/_common.scss b/data/theme/gnome-shell-sass/_common.scss
index 64783486a..d21855787 100644
--- a/data/theme/gnome-shell-sass/_common.scss
+++ b/data/theme/gnome-shell-sass/_common.scss
@@ -155,14 +155,17 @@ stage {
 
 
 // notification styling
-%notification_bubble {
+@mixin notification_bubble($flat: false) {
   border-width: 1px;
   border-style: solid;
-  border-radius:$base_border_radius + 2px;
-  padding: 0;
+  border-radius: $base_border_radius + 2px;
   margin: $base_margin;
 
-  @include button(normal);
+  @if $flat {
+    @include button(undecorated);
+  } @else {
+    @include button(normal);
+  }
 
   &:focus {
     @include button(focus);
diff --git a/data/theme/gnome-shell-sass/widgets/_calendar.scss b/data/theme/gnome-shell-sass/widgets/_calendar.scss
index 0331dd33f..fa1e75fe8 100644
--- a/data/theme/gnome-shell-sass/widgets/_calendar.scss
+++ b/data/theme/gnome-shell-sass/widgets/_calendar.scss
@@ -21,17 +21,8 @@
 
   // today button (the date)
   .datemenu-today-button {
+    @include notification_bubble($flat: true);
     padding: $base_padding * 1.5;
-    margin: $base_margin;
-    border: 1px solid transparent;
-    border-radius: $base_border_radius + 2px;
-
-    &:hover { @include button(hover);}
-    &:focus { @include button(focus);}
-
-    &:active {
-      @include button(active);
-    }
 
     // weekday label
     .day-label {
@@ -48,8 +39,7 @@
 
   // calendar
   .calendar {
-    @extend %notification_bubble;
-    margin: $base_margin;
+    @include notification_bubble;
     margin-bottom: $base_padding + $base_margin;
     padding: $base_padding;
 
@@ -67,7 +57,7 @@
     // world clocks and weather
     .world-clocks-button,
     .weather-button {
-      @extend %notification_bubble;
+      @include notification_bubble;
       padding: $base_padding;
     }
   }
diff --git a/data/theme/gnome-shell-sass/widgets/_message-list.scss b/data/theme/gnome-shell-sass/widgets/_message-list.scss
index adedcf71d..9893b6897 100644
--- a/data/theme/gnome-shell-sass/widgets/_message-list.scss
+++ b/data/theme/gnome-shell-sass/widgets/_message-list.scss
@@ -32,7 +32,7 @@
 
 // message bubbles
 .message {
-  @extend %notification_bubble;
+  @include notification_bubble;
 
   // title
   .message-title {