From 37e5b31408a0258cecea414ffe5b783690f15f1e Mon Sep 17 00:00:00 2001 From: Sam Hewitt Date: Tue, 24 Sep 2024 13:01:27 -0230 Subject: [PATCH] style: Saner padding and margins in notification headers Closes: https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/7920 Part-of: --- .../widgets/_message-list.scss | 20 +++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/data/theme/gnome-shell-sass/widgets/_message-list.scss b/data/theme/gnome-shell-sass/widgets/_message-list.scss index 60cb5bde7..26cfb4537 100644 --- a/data/theme/gnome-shell-sass/widgets/_message-list.scss +++ b/data/theme/gnome-shell-sass/widgets/_message-list.scss @@ -74,9 +74,9 @@ color: $insensitive_fg_color; padding:0 $base_padding; - // side padding to accommodate the close button - &:ltr { padding-right:$base_padding - 2px; }; - &:rtl { padding-left:$base_padding - 2px; }; + // remove side padding to accommodate the close button + &:ltr { padding-right: 0; } + &:rtl { padding-left: 0; } // header source icon .message-source-icon { @@ -102,8 +102,8 @@ // Add bottom padding to align the app name with the time horizontally padding-bottom: to_em(1px); - &:ltr { text-align: right }; - &:rtl { text-align: left }; + &:ltr { text-align: right; } + &:rtl { text-align: left; } } } @@ -114,13 +114,17 @@ @extend %notification_button; border-radius: $forced_circular_radius; // make circular color: $fg_color; - padding: 4px; } + // increase padding to expand button to make it larger .message-expand-button { padding: $base_padding; - &:ltr { margin-right: $base_padding; } - &:rtl { margin-left: $base_padding; } + } + + // add margin to close button to balance it with the expand button + .message-close-button { + margin: $base_padding * 0.5; + padding: 4px; } }