Make hoverEnabled propagate to children

Hover effects can be sometimes a bit disturbing. This change makes it
possible to conveniently disable hover effects for a tree of controls
in one go. For example, to disable hover effects for a page including
its header, footer, and all list items:

    Page {
        hoverEnabled: false
        header: ...
        footer: ...
        ListView {
            delegate: ...
        }
    }

[ChangeLog][Controls][Important Behavior Changes] Control::hoverEnabled
has been made to inherit to children, to make it possible to disable
hover effects for a tree of controls in one place.

Change-Id: Ia87144f2cc04957a32f89d3313816b91d97db635
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>

1 files changed, 39 insertions, 1 deletions

diff --git a/tests/auto/controls/data/tst_control.qml b/tests/auto/controls/data/tst_control.qml
index 77f23fa5..f07f6051 100644
--- a/tests/auto/controls/data/tst_control.qml
+++ b/tests/auto/controls/data/tst_control.qml
@@ -852,7 +852,9 @@ TestCase {
         verify(control)
 
         compare(control.hovered, false)
-        compare(control.hoverEnabled, false)
+        compare(control.hoverEnabled, Qt.styleHints.useHoverEffects)
+
+        control.hoverEnabled = false
 
         mouseMove(control, control.width / 2, control.height / 2)
         compare(control.hovered, false)
@@ -874,6 +876,42 @@ TestCase {
         control.destroy()
     }
 
+    function test_hoverEnabled() {
+        var control = component.createObject(testCase)
+        compare(control.hoverEnabled, Qt.styleHints.useHoverEffects)
+
+        var child = component.createObject(control)
+        var grandChild = component.createObject(child)
+
+        var childExplicitHoverEnabled = component.createObject(control, {hoverEnabled: true})
+        var grandChildExplicitHoverDisabled = component.createObject(childExplicitHoverEnabled, {hoverEnabled: false})
+
+        var childExplicitHoverDisabled = component.createObject(control, {hoverEnabled: false})
+        var grandChildExplicitHoverEnabled = component.createObject(childExplicitHoverDisabled, {hoverEnabled: true})
+
+        control.hoverEnabled = false
+        compare(control.hoverEnabled, false)
+        compare(grandChild.hoverEnabled, false)
+
+        compare(childExplicitHoverEnabled.hoverEnabled, true)
+        compare(grandChildExplicitHoverDisabled.hoverEnabled, false)
+
+        compare(childExplicitHoverDisabled.hoverEnabled, false)
+        compare(grandChildExplicitHoverEnabled.hoverEnabled, true)
+
+        control.hoverEnabled = true
+        compare(control.hoverEnabled, true)
+        compare(grandChild.hoverEnabled, true)
+
+        compare(childExplicitHoverEnabled.hoverEnabled, true)
+        compare(grandChildExplicitHoverDisabled.hoverEnabled, false)
+
+        compare(childExplicitHoverDisabled.hoverEnabled, false)
+        compare(grandChildExplicitHoverEnabled.hoverEnabled, true)
+
+        control.destroy()
+    }
+
     function test_implicitSize() {
         var control = component.createObject(testCase)
         verify(control)