Control: fix regression where hoverEnabled defaulted to false

Now (after 83d9084) that
QQuickContentItem's parent is the content control rather
than QQuickRootItem, we need to ensure that the control has the
correct hoverEnabled value. If we don't do this (get the value
from the style hints), it will use QQuickItem's default value of false
and some controls won't be hoverable when they should be.

Fixes: QTBUG-133886
Pick-to: 6.9
Change-Id: Ib0912acf7cdf513bf004837bc4ede970decf23d8
Reviewed-by: Tor Arne Vestbø <[email protected]>

Author: mitchcurtis

src/quicktemplates/qquickapplicationwindow.cpp

@@ -822,6 +822,13 @@ void QQuickApplicationWindow::classBegin()
     // them.
     d->control = new QQuickControl(QQuickWindow::contentItem());
     d->control->setObjectName("ApplicationWindowContentControl");
+    // Now that QQuickContentItem's parent is the content control rather
+    // than QQuickRootItem, we need to ensure that the control has the
+    // correct hoverEnabled value. If we don't do this (get the value
+    // from the style hints), it will use QQuickItem's default value of false
+    // and some controls won't be hoverable when they should be.
+    QQuickControlPrivate::get(d->control)->updateHoverEnabled(
+        QQuickControlPrivate::calcHoverEnabled(nullptr), false);
     auto *contentItem = new QQuickContentItem(this, d->control);
     // The content item can't be its own focus scope here, as that
     // will detach focus of items inside the content item from focus

tests/auto/quickcontrols/qquickcontrol/data/controlInApplicationWindow.qml

@@ -0,0 +1,18 @@
+// Copyright (C) 2025 The Qt Company Ltd.
+// SPDX-License-Identifier: LicenseRef-Qt-Commercial OR GPL-3.0-only
+
+import QtQuick
+import QtQuick.Controls
+
+ApplicationWindow {
+    width: 400
+    height: 400
+
+    property alias control: control
+
+    Control {
+        id: control
+        width: 100
+        height: 100
+    }
+}

tests/auto/quickcontrols/qquickcontrol/tst_qquickcontrol.cpp

@@ -27,6 +27,7 @@ private slots:
     void fractionalFontSize();
     void resizeBackgroundKeepsBindings();
     void hoverInMouseArea();
+    void hoverEnabledDefault();
 
 private:
     QScopedPointer<QPointingDevice> touchDevice;
@@ -137,6 +138,18 @@ void tst_QQuickControl::hoverInMouseArea()
     QVERIFY(textField->isHovered());
 }
 
+void tst_QQuickControl::hoverEnabledDefault()
+{
+    QQuickApplicationHelper helper(this, QStringLiteral("controlInApplicationWindow.qml"));
+    QQuickWindow *window = helper.window;
+    window->show();
+    QVERIFY(QTest::qWaitForWindowExposed(window));
+
+    const auto *control = window->property("control").value<QQuickControl *>();
+    QVERIFY(control);
+    QCOMPARE(control->isHoverEnabled(), QGuiApplication::styleHints()->useHoverEffects());
+}
+
 QTEST_QUICKCONTROLS_MAIN(tst_QQuickControl)
 
 #include "tst_qquickcontrol.moc"