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

Regression caused by the fix for QTBUG-16252

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • P1: Critical
    • 5.9.7
    • 5.9.6
    • None
    • Should be platform agnostic.
    • 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

          For Gerrit Dashboard: QTBUG-68939
          # Subject Branch Project Status CR V

          Activity

            People

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

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There is 1 open Gerrit change