diff options
| author | Jan Arve Saether <jan-arve.saether@digia.com> | 2013-03-13 08:58:54 +0100 |
|---|---|---|
| committer | The Qt Project <gerrit-noreply@qt-project.org> | 2013-03-14 16:25:44 +0100 |
| commit | 0c935730b2cdf459567cf4c85872ca93ad31fa62 (patch) | |
| tree | 0e628f35bc5ded12e8afdc819e9c7cb34f78f7d1 /src/controls/qtmenu.cpp | |
| parent | 5145aec470344b43669a6e1e68ccade3886d5351 (diff) | |
Rearrange only the topmost layout.
Since we also rearrange in geometryChanged(), this should ensure
that we recurse down and arrange all sublayouts.
This assumes that objects are constructed in bottom-up order.
(That's what componentComplete() indicates.)
Previously we would risk rearranging sublayouts several times:
1. Sublayout got componentComplete -> geometryChanged -> rearrange()
2. Parent layout got componentComplete -> geometryChanged -> rearrange()
3. Since the sublayout was a layout item of the parent layout it could
get a geometry change again, causing it to rearrange again.
This also fixes the issue where the implicitWidth/Height of the layout
was not immediately set after componentComplete, which could cause it
to get the wrong initial size.
Change-Id: I63a35dd934cd3ace01fab6319333d531631a6f4e
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>
Diffstat (limited to 'src/controls/qtmenu.cpp')
0 files changed, 0 insertions, 0 deletions