Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-68939

Regression caused by the fix for QTBUG-16252

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: P1: Critical
    • Resolution: Done
    • Affects Version/s: 5.9.6
    • Fix Version/s: 5.9.7
    • Labels:
      None
    • Environment:
      Should be platform agnostic.
    • Commits:
      eaa2e164b0559b37dbcd060cd31e110d2775ffd6

      Description

      The fix for QTBUG-16252 introduced a regression which I have so far only seen in KDevelop.

       

      There, document views behave as if hidden dock widgets are in fact expanded, with the result that they are too narrow and not high enough. Expanding a dock widget (on any edge) and changing its width or height (or changing the width/height of an already expanded dock widget) restores the proper doc view dimensions. This fix persists when the dock widget is hidden again (= the doc view takes up all available space) but not when I change to another doc view (in KDevelop's tabbed interface).

       

      Resizing the entire window doesn't help; doc views and the unoccupied space are resized proportionally.

       

      This issue disappears when I revert commit #cbfb6bda1d1ce3e169db6a0deb9bd901076653e4 .

        Attachments

        1. docks.png
          docks.png
          122 kB
        2. dolphin-18.04.2_with_qt-5.9.6.png
          dolphin-18.04.2_with_qt-5.9.6.png
          60 kB
        3. QMainWindowX.cpp
          3 kB
        4. QMainWindowX.h
          1.0 kB
        5. qt596-qtbug16252-regression-glitch.png
          qt596-qtbug16252-regression-glitch.png
          49 kB

          Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            Activity

              People

              • Assignee:
                qt.team.quick.subscriptions Qt Quick and Widgets Team
                Reporter:
                rjvbertin René Bertin
              • Votes:
                12 Vote for this issue
                Watchers:
                19 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Gerrit Reviews

                  There is 1 open Gerrit change