[vlc-commits] qml: revert table view to use 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| [bc724435863ad842a04eb42e2045905b3a042f61] | committer: Jean-Baptiste Kempf

qml: revert table view to use 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=bc724435863ad842a04eb42e2045905b3a042f61
---

 modules/gui/qt/qml/utils/KeyNavigableTableView.qml | 23 ++++++----------------
 1 file changed, 6 insertions(+), 17 deletions(-)

diff --git a/modules/gui/qt/qml/utils/KeyNavigableTableView.qml b/modules/gui/qt/qml/utils/KeyNavigableTableView.qml
index 5e0aa8bd2e..9270f2d622 100644
--- a/modules/gui/qt/qml/utils/KeyNavigableTableView.qml
+++ b/modules/gui/qt/qml/utils/KeyNavigableTableView.qml
@@ -67,7 +67,7 @@ NavigableFocusScope {
 
                 width: root.width
                 height: root.rowHeight
-                color: VLCStyle.colors.bg
+                color: VLCStyle.colors.getBgColor(element.DelegateModel.inSelected, hoverArea.containsMouse, lineView.activeFocus)
 
                 MouseArea {
                     id: hoverArea
@@ -89,24 +89,13 @@ NavigableFocusScope {
                     onDoubleClicked: {
                         actionForSelection(delegateModel.selectedGroup)
                     }
-                    RectangularGlow {
-                        visible: element.DelegateModel.inSelected || hoverArea.containsMouse
-                        anchors.fill: parent
-                        glowRadius: VLCStyle.margin_xxsmall
-                        color: VLCStyle.colors.getBgColor(element.DelegateModel.inSelected, hoverArea.containsMouse, lineView.activeFocus)
-                    }
-
-                    Rectangle{
-                        anchors.fill: parent
-                        anchors.topMargin: VLCStyle.margin_xxsmall
-                        anchors.bottomMargin: VLCStyle.margin_xxsmall
-                        anchors.leftMargin: VLCStyle.margin_xxxsmall
-                        anchors.rightMargin: VLCStyle.margin_xxxsmall
-                        color: VLCStyle.colors.bg
-                        radius: height / 10
 
                         Row {
                             anchors {
+                                topMargin: VLCStyle.margin_xxsmall
+                                bottomMargin: VLCStyle.margin_xxsmall
+                                leftMargin: VLCStyle.margin_xxxsmall
+                                rightMargin: VLCStyle.margin_xxxsmall
                                 fill: parent
                             }
                             Repeater {
@@ -128,7 +117,7 @@ NavigableFocusScope {
                                 }
                             }
                         }
-                    }
+
                 }
             }
         }