[vlc-commits] qml: revert grid items to use a solid background instead of a glowing frame

[Pierre Lamot]

vlc | branch: master | Pierre Lamot <pierre at videolabs.io> | Wed Sep 25 16:48:53 2019 +0200| [bbc1d1ab95073288812bf7912d3c00fdcc674afc] | committer: Jean-Baptiste Kempf

qml: revert  grid items to use a solid background instead of a glowing frame

Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=bbc1d1ab95073288812bf7912d3c00fdcc674afc
---

 modules/gui/qt/qml/utils/GridItem.qml | 17 +++++------------
 1 file changed, 5 insertions(+), 12 deletions(-)

diff --git a/modules/gui/qt/qml/utils/GridItem.qml b/modules/gui/qt/qml/utils/GridItem.qml
index 755694df61..1cbb760989 100644
--- a/modules/gui/qt/qml/utils/GridItem.qml
+++ b/modules/gui/qt/qml/utils/GridItem.qml
@@ -56,11 +56,10 @@ Rectangle {
 
     property int index: 0
 
-    Rectangle {
+    Item {
         id: gridItem
         width: childrenRect.width
         height: childrenRect.height
-        color: "transparent"
         
         MouseArea {
             id: mouseArea
@@ -91,17 +90,11 @@ Rectangle {
                     id: cover_bg
                     width: picture.width
                     height: picture.height
-                    color: (cover.status !== Image.Ready) ? VLCStyle.colors.banner : "transparent"
 
-                    RectangularGlow {
-                        visible: picture.highlighted || mouseArea.containsMouse
-                        anchors.fill: cover
-                        spread: 0.1
-                        glowRadius: VLCStyle.margin_xxsmall
-                        color: VLCStyle.colors.getBgColor(
-                                   selected, mouseArea.containsMouse,
-                                   root.activeFocus)
-                    }
+
+                    color: VLCStyle.colors.getBgColor(
+                               selected, mouseArea.containsMouse,
+                               root.activeFocus)
 
                     RoundImage {
                         id: cover