[qt/qt/qtdeclarative]: Summary of bulk changes made
KDE Git Services - Bulk Change <[email protected]>
| Newsgroups | gmane.comp.kde.cvs |
|---|---|
| Message-ID | <[email protected]> |
Git repository change summary for qt/qt/qtdeclarative Pushed by mirror-service into branch '6.12'. Changed from 964c27f3ae40dc739a57d063b8daf593466f301a to 0a8bacd781c5a71d626573b07ba7bae3b7708fad Acknowledgement was received that this change introduces only existing code that has been pushed to another public open source repository. This change contains the following new commits: Git commit 0a8bacd781c5a71d626573b07ba7bae3b7708fad by Qt Cherry-pick Bot (on behalf of SanthoshKumar Selvaraj) on 21/07/2026 at 17:33.. Popup: Fix test_margins to use overlay instead of testCase size test_margins() computed the expected popup position using testCase.width/height. The testCase size is the requested geometry, but not the actual window geometry, so they stay fixed at the values declared in the test file. QQuickPopupPositioner, places the popup against the actual geometry of Overlay.overlay. On platforms where the real window ends up a different size than what the application requested (e.g. OHOS, which forces the window to an OS-controlled fullscreen size minus status bar/nav bar insets, regardless of the requested QML width/height), testCase.width/height diverges from the overlay's actual size, and the expected values no longer match. Fix by reading control.Overlay.overlay width/height instead of testCase declared size when computing the expected positions. This is not an issue on platforms where window and overlay size already match the requested size. Also, remove the blacklist in android as well. Task-number: QTBUG-146901 Change-Id: I725bac01731e4c36f3377d7d7cb6842cfd6c1b58 Reviewed-by: Assam Boudjelthia <[email protected]> (cherry picked from commit 633450dea5791f199c37afcd2cbd556574ad31ed) Reviewed-by: Qt Cherry-pick Bot <[email protected]> https://invent.kde.org/qt/qt/qtdeclarative/-/commit/0a8bacd781c5a71d626573b07ba7bae3b7708fad