ScrollBar: use contentItem instead of handle

Thanks to virtual resizeContent() we can finally use contentItem like
everywhere else, and cleanup the special handle item.

Change-Id: I9e52ec3ff327d19cda3d85d2e2d9b73d80442bbc
Reviewed-by: Mitch Curtis <mitch.curtis@theqtcompany.com>

1 files changed, 5 insertions, 11 deletions

diff --git a/src/imports/controls/ScrollBar.qml b/src/imports/controls/ScrollBar.qml
index 55d53191..16ac2dfc 100644
--- a/src/imports/controls/ScrollBar.qml
+++ b/src/imports/controls/ScrollBar.qml
@@ -41,14 +41,14 @@ T.ScrollBar {
     id: control
 
     implicitWidth: Math.max(background ? background.implicitWidth : 0,
-                            handle.implicitWidth + leftPadding + rightPadding)
+                            contentItem.implicitWidth + leftPadding + rightPadding)
     implicitHeight: Math.max(background ? background.implicitHeight : 0,
-                             handle.implicitHeight + topPadding + bottomPadding)
+                             contentItem.implicitHeight + topPadding + bottomPadding)
 
     padding: 2
 
-    //! [handle]
-    handle: Rectangle {
+    //! [contentItem]
+    contentItem: Rectangle {
         id: handle
 
         implicitWidth: 6
@@ -59,12 +59,6 @@ T.ScrollBar {
         visible: control.size < 1.0
         opacity: 0.0
 
-        readonly property bool horizontal: control.orientation === Qt.Horizontal
-        x: control.leftPadding + (horizontal ? control.position * control.width : 0)
-        y: control.topPadding + (horizontal ? 0 : control.position * control.height)
-        width: horizontal ? control.size * control.availableWidth : implicitWidth
-        height: horizontal ? implicitHeight : control.size * control.availableHeight
-
         states: State {
             name: "active"
             when: control.active
@@ -79,5 +73,5 @@ T.ScrollBar {
             }
         }
     }
-    //! [handle]
+    //! [contentItem]
 }