[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.11'.
Changed from 0aba8c7674dbbbaaa142775247249e297a339c85 to ca6021de39756b53e26ed08da3d0e8f7bb819013
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 ca6021de39756b53e26ed08da3d0e8f7bb819013 by Oliver Eftevaag on 06/08/2026 at 11:16..
PopupWindow: prioritize explicitly set width/height values
Explicit bindings on width and height, aka:
```
Popup {
width: 500
height: 500
}
```
should take priority over the implicit size that's derived from the
first child Item's size.
For popups where the popupType is Popup.Window, the popup window would
react to changes in the first child's implicit size with equal priority
to its own explicitly given size. This made it difficult to control the
size of popup windows, and lead to surprising bugs.
Fix it by skipping setWidth()/setHeight() in
implicitWidthChanged()/implicitHeightChanged() if the hasWidth/hasHeight
flag is set.
Fixes: QTBUG-147847
Pick-to: 6.8
Change-Id: I542e0a90b934b0eee7b5cab4370f890c2b6ca755
Reviewed-by: Mitch Curtis <[email protected]>
Reviewed-by: MohammadHossein Qanbari <[email protected]>
(cherry picked from commit 2fadac0fb9d3cefbc25706e4f2483d11730bfe57)
Reviewed-by: Qt Cherry-pick Bot <[email protected]>
(cherry picked from commit fb29854b29436362565998105f65a4a0e6b265a0)
https://invent.kde.org/qt/qt/qtdeclarative/-/commit/ca6021de39756b53e26ed08da3d0e8f7bb819013