Ubuntu
unity8 package

QDockWidgets don't work at all

Bug #1673415 reported by Albert Astals Cid
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
qtubuntu (Ubuntu)
Fix Released
Undecided
Daniel d'Andrada
unity8 (Ubuntu)
Triaged
Undecided
Unassigned

Bug Description

How to reproduce:
 * Install tiled
 * Start tiled
 * Press the small square button next to properties
 * That should undock the properties widget, make it a window and you should be able to drag it freely around the desktop, that doesn't happen
 * Alternatively, drag the properties widget from the "properties" text, you'll be able to drag it, but only inside the window, instead of around the desktop, and once you release it, you can't drag it anymore

To "reset" the docked status, remove ~/.config/mapeditor.org/tiled.conf

Related branches

lp://qastaging/~dandrader/qtubuntu/qtToolIsMirSatellite
Albert Astals Cid (community): Approve
Unity8 CI Bot: Approve (continuous-integration)
Diff: 30 lines (+11/-1)
1 file modified
src/ubuntumirclient/qmirclientwindow.cpp (+11/-1)
lp://qastaging/~dandrader/unity8/satelliteWindow
Unity8 CI Bot: Approve (continuous-integration)
Albert Astals Cid (community): Approve
Diff: 31 lines (+6/-1)
1 file modified
qml/Stage/ChildWindow.qml (+6/-1)
Daniel d'Andrada (dandrader)
Changed in unity8 (Ubuntu):
assignee: nobody → Daniel d'Andrada (dandrader)
status: New → In Progress
Daniel d'Andrada (dandrader)
Changed in qtubuntu (Ubuntu):
assignee: nobody → Daniel d'Andrada (dandrader)
status: New → In Progress
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

We are missing support for satellite windows.

summary: - QDockWidgets don't work at all
+ Missing support for satellite (Qt::Tool) windows
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

This actually goes deeper than simply supporting satellite window types.

summary: - Missing support for satellite (Qt::Tool) windows
+ QDockWidgets don't work at all
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

Other issues being:
1 - we are forcibly killing windows that are closed. it's perfectly ok for an application to simply hide a window whose close button was clicked.

2 - this whole drag and drop behavior/interaction between the dockable satellite/tool and the main window is still to be investigated.

Revision history for this message
Daniel d'Andrada (dandrader) wrote :

One of the prerequisite fixes is bug 1671424

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.15+17.04.20170328.3-0ubuntu1

---------------
unity8 (8.15+17.04.20170328.3-0ubuntu1) zesty; urgency=medium

  [ Albert Astals Cid ]
  * mock: Remove empty property and move invokable
  * Tune quicklist item selection
  * Fix unminimizing going to the wrong position the second time (LP:
    #1671458)
  * Remove unclickable gaps on the menubar (LP: #1672722)
  * indicators client is a tool
  * Make sure we destroy the popups when the item goes away (LP:
    #1670338)
  * UnityApplicationMocks: Initialize m_state and m_requestedState
  * Don't use appInfo after checking it if is null
  * Initialize m_privateMode
  * Hook up aboutToShow for overflow menus (LP: #1676016)

  [ Daniel d'Andrada ]
  * Give active focus to child surface qml items (LP: #1671072)
  * Satellite child windows (LP: #1673415)
  * Tell qtmir/miral about the available desktop area

  [ Florian Boucault ]
  * WindowStateStorage: use a private QThreadPool to ensure that
    WindowStateStorage::saveValue always has a thread available to
    execute the query in. (LP: #1675424)

  [ Lukáš Tinkl ]
  * Protect against loading invalid window geometry (LP: #1674262)
  * Remove usage of the deprecated (and non-functional) system-image-
    dbus service.
  * Implement Ctrl+Alt+T to launch the terminal app (LP: #1673500)
  * Only take normal and dialog surface types into account for launcher
    items (LP: #1669047)
  * Fix the shutdown dialog after recent refactoring in
    DBusUnitySessionService (LP: #1676426)

  [ Michael Zanetti ]
  * drop the grey background behind icons in the drawer (LP: #1675688)

  [ Pete Woods ]
  * Fix Unity/Platform::isPC, and add tests (LP: #1670657)

 -- Michał Sawicz <email address hidden> Tue, 28 Mar 2017 21:50:20 +0000

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtubuntu - 0.64+17.04.20170328.1-0ubuntu1

---------------
qtubuntu (0.64+17.04.20170328.1-0ubuntu1) zesty; urgency=medium

  [ Albert Astals Cid ]
  * Remove signal noone uses
  * Honor the menu item visibility (LP: #1675802)

  [ Alberto Aguirre, Arthur Mello ]
  * Remove more uses of deprecated mir apis.

  [ Daniel d'Andrada ]
  * Qt::Tool == mir_window_type_satellite (LP: #1673415)
  * Proper implementation of QPlatformScreen::logicalDpi (LP: #1675572)

  [ Gerry Boland ]
  * BackingStore: is possible for context's QPlatformSurface to be
    deleted before backing store is.

 -- Michał Sawicz <email address hidden> Tue, 28 Mar 2017 17:13:20 +0000

Changed in qtubuntu (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

Latest landings improved the situation but this bug is not completely fixed yet.

Changed in unity8 (Ubuntu):
status: Fix Released → Triaged
assignee: Daniel d'Andrada (dandrader) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.