Details
-
Epic
-
Resolution: Unresolved
-
P2: Important
-
None
-
None
-
None
-
Enable PopupWindows by default
-
876131de6 (dev), c04dd4ca7 (6.8), b1599c148 (dev), b00ed190c (dev), 0475b5b08 (6.8), d758bdc9c (6.8), 09518529a (dev), 70f937167 (dev), 622a77041 (6.8), 5443df4f1 (6.8), 1d59ba0a6 (dev), 4e13d7a28 (6.8), b5d379d28 (dev), 3eff6a639 (dev), 643eefc5a (dev)
Description
Now that we have the popupType property in QQuickPopup, it would be nice to enable popup windows by default.
If the underlying platform can't use popup windows, it will fall back to popup items anyways. So might as well try to use popup windows as the default value.
Unfortunately, there are a lot of tests that will need to be rewritten, to take into account some of the changes that are in effect for popup windows.
Such as, waiting for the window to be active and polished, before sending events to the window.
In some cases, especially for Dialogs, the popup window will need to be the receiver of QTest::keyClick() and QTest::mouseClick() type events.
And tests that are checking the Overlay's children, won't find popups that use popup windows.
These are things to keep in mind when going trough the following list of tests:
tst_basic
tst_fusion
tst_imagine
tst_ios
tst_macos
tst_material
tst_windows
Ed:
tst_focus (2 failing tests: policy(ComboBox), reason())
Oliver:
tst_qquickmenu (1 failing tests...)
The tests above, are suspected to not work, since they've failed in the past, when we changed the default.
The exact test functions will need to be discovered again, as we go through each of those test classes.