From fe3bf3a63885e24706772876778e8f11f13788f3 Mon Sep 17 00:00:00 2001 From: Joshua Goins Date: Wed, 26 Jul 2023 01:47:56 -0400 Subject: [PATCH] Pick a better shadow and clean up emoji popup backgrounds These shadows are now closer to how other popups look. --- src/qml/Component/Emoji/EmojiTonesPicker.qml | 15 ++++++++++----- src/qml/Dialog/EmojiDialog.qml | 15 ++++++++++----- 2 files changed, 20 insertions(+), 10 deletions(-) diff --git a/src/qml/Component/Emoji/EmojiTonesPicker.qml b/src/qml/Component/Emoji/EmojiTonesPicker.qml index 2e37b80d2..f67f4fd7b 100644 --- a/src/qml/Component/Emoji/EmojiTonesPicker.qml +++ b/src/qml/Component/Emoji/EmojiTonesPicker.qml @@ -30,11 +30,16 @@ QQC2.Popup { onOpened: x = Math.min(parent.mapFromGlobal(QQC2.Overlay.overlay.width - tones.width, 0).x, -(width - parent.width) / 2) background: Kirigami.ShadowedRectangle { color: Kirigami.Theme.backgroundColor - radius: Kirigami.Units.smallSpacing - shadow.size: Kirigami.Units.smallSpacing - shadow.color: Qt.rgba(0.0, 0.0, 0.0, 0.10) - border.color: Kirigami.ColorUtils.tintWithAlpha(color, Kirigami.Theme.textColor, 0.15) - border.width: 1 + radius: Kirigami.Units.mediumSpacing + shadow { + size: Kirigami.Units.largeSpacing + color: Qt.rgba(0.0, 0.0, 0.0, 0.3) + yOffset: 2 + } + border { + color: Kirigami.ColorUtils.tintWithAlpha(color, Kirigami.Theme.textColor, 0.15) + width: 1 + } } ListView { diff --git a/src/qml/Dialog/EmojiDialog.qml b/src/qml/Dialog/EmojiDialog.qml index 73e388b1a..91651c8f9 100644 --- a/src/qml/Dialog/EmojiDialog.qml +++ b/src/qml/Dialog/EmojiDialog.qml @@ -39,11 +39,16 @@ QQC2.Popup { background: Kirigami.ShadowedRectangle { Kirigami.Theme.colorSet: Kirigami.Theme.View color: Kirigami.Theme.backgroundColor - radius: Kirigami.Units.smallSpacing - shadow.size: Kirigami.Units.smallSpacing - shadow.color: Qt.rgba(0.0, 0.0, 0.0, 0.10) - border.color: Kirigami.ColorUtils.tintWithAlpha(color, Kirigami.Theme.textColor, 0.15) - border.width: 2 + radius: Kirigami.Units.mediumSpacing + shadow { + size: Kirigami.Units.largeSpacing + color: Qt.rgba(0.0, 0.0, 0.0, 0.3) + yOffset: 2 + } + border { + color: Kirigami.ColorUtils.tintWithAlpha(color, Kirigami.Theme.textColor, 0.15) + width: 2 + } } modal: true