[convergence] QWindow maximum and minimum values for width and height are not respected
Bug #1511530 reported by
Gustavo Pichorim Boiko
This bug affects 3 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical Pocket Desktop |
Fix Released
|
Medium
|
kevin gunn | ||
| Canonical System Image |
Fix Released
|
Medium
|
kevin gunn | ||
| Ubuntu UX |
Fix Released
|
Medium
|
Olga Kemmet | ||
| qtmir (Ubuntu) |
Fix Released
|
Medium
|
Daniel d'Andrada | ||
| unity8 (Ubuntu) |
Fix Released
|
Undecided
|
Daniel d'Andrada | ||
Bug Description
When trying to set constraints to a QWindow-based application using maximumWidth, maximumHeight, minimumWidth, minimumHeight, those values are not respected by unity8's window manager.
----------- UX comment -----------
This bug is somewhat related to #1488852, as per comment #1. This issue can affect other apps, apart from the dialer and developers need to have the ability to set window values.
Related branches
lp:~dandrader/unity8/sizeHints
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Albert Astals Cid (community): Abstain (merges fine)
- Lukáš Tinkl (community): Approve
-
Diff: 1043 lines (+651/-80)15 files modifiedCMakeLists.txt (+1/-1)
debian/control (+3/-3)
qml/Stages/ApplicationWindow.qml (+7/-0)
qml/Stages/DecoratedWindow.qml (+7/-0)
qml/Stages/DesktopStage.qml (+6/-0)
qml/Stages/WindowResizeArea.qml (+95/-16)
tests/mocks/Unity/Application/MirSurface.cpp (+48/-0)
tests/mocks/Unity/Application/MirSurface.h (+21/-0)
tests/mocks/Unity/Application/SurfaceManager.cpp (+56/-0)
tests/mocks/Unity/Application/SurfaceManager.h (+40/-0)
tests/qmltests/Stages/SizeHintField.qml (+33/-0)
tests/qmltests/Stages/SurfaceManagerControls.qml (+33/-0)
tests/qmltests/Stages/SurfaceManagerField.qml (+40/-0)
tests/qmltests/Stages/tst_DesktopStage.qml (+36/-30)
tests/qmltests/Stages/tst_WindowResizeArea.qml (+225/-30)
lp:~dandrader/unity-api/surfaceItemSizeHints
- Lukáš Tinkl (community): Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 99 lines (+56/-1)3 files modifieddebian/changelog (+6/-0)
include/unity/shell/application/CMakeLists.txt (+1/-1)
include/unity/shell/application/MirSurfaceInterface.h (+49/-0)
lp:~dandrader/qtmir/sizeHints
- Lukáš Tinkl (community): Approve
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
-
Diff: 892 lines (+392/-47)20 files modifiedCMakeLists.txt (+1/-1)
debian/control (+2/-2)
src/modules/Unity/Application/mirsurface.cpp (+93/-1)
src/modules/Unity/Application/mirsurface.h (+26/-1)
src/modules/Unity/Application/mirsurfaceinterface.h (+7/-0)
src/modules/Unity/Application/mirsurfacemanager.cpp (+4/-2)
src/modules/Unity/Application/mirsurfacemanager.h (+4/-1)
src/platforms/mirserver/CMakeLists.txt (+1/-0)
src/platforms/mirserver/mirserver.cpp (+3/-2)
src/platforms/mirserver/mirwindowmanager.cpp (+23/-6)
src/platforms/mirserver/mirwindowmanager.h (+5/-2)
src/platforms/mirserver/sessionlistener.cpp (+11/-1)
src/platforms/mirserver/sessionlistener.h (+10/-1)
src/platforms/mirserver/sizehints.cpp (+44/-0)
src/platforms/mirserver/sizehints.h (+52/-0)
src/platforms/mirserver/surfaceobserver.cpp (+58/-1)
src/platforms/mirserver/surfaceobserver.h (+25/-0)
tests/mirserver/WindowManager/window_manager.cpp (+2/-22)
tests/modules/SurfaceManager/mirsurface_test.cpp (+7/-4)
tests/modules/common/fake_mirsurface.h (+14/-0)
| Changed in canonical-pocket-desktop: | |
| assignee: | nobody → kevin gunn (kgunn72) |
| Changed in qtmir: | |
| importance: | Undecided → Medium |
| Changed in canonical-pocket-desktop: | |
| importance: | Undecided → Medium |
| Changed in qtubuntu: | |
| importance: | Undecided → Medium |
| Changed in canonical-devices-system-image: | |
| assignee: | nobody → kevin gunn (kgunn72) |
| milestone: | none → backlog |
Revision history for this message
| Bill Filler (bfiller) wrote | #1 |
| Changed in canonical-devices-system-image: | |
| status: | New → Confirmed |
| importance: | Undecided → Medium |
| Changed in qtubuntu: | |
| assignee: | nobody → Daniel d'Andrada (dandrader) |
| Changed in qtmir: | |
| assignee: | nobody → Daniel d'Andrada (dandrader) |
| status: | New → In Progress |
| Changed in qtubuntu: | |
| status: | New → In Progress |
| Changed in canonical-devices-system-image: | |
| status: | Confirmed → In Progress |
| milestone: | backlog → ww02-2016 |
| Changed in unity8: | |
| assignee: | nobody → Daniel d'Andrada (dandrader) |
| importance: | Undecided → Medium |
| status: | New → In Progress |
| Changed in qtubuntu: | |
| status: | In Progress → Fix Released |
| summary: |
- QWindow maximum and minimum values for width and height are not - respected + [convergence] QWindow maximum and minimum values for width and height + are not respected |
| Changed in ubuntu-ux: | |
| status: | New → Fix Committed |
| importance: | Undecided → Medium |
| assignee: | nobody → Olga Kemmet (olga-kemmet) |
| assignee: | Olga Kemmet (olga-kemmet) → nobody |
| assignee: | nobody → Benjamin Keyser (bjkeyser) |
| description: | updated |
| Changed in ubuntu-ux: | |
| assignee: | Benjamin Keyser (bjkeyser) → Olga Kemmet (olga-kemmet) |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #2 |
| Changed in qtmir (Ubuntu): | |
| status: | New → Confirmed |
| Changed in qtubuntu (Ubuntu): | |
| status: | New → Confirmed |
| Changed in unity8 (Ubuntu): | |
| status: | New → Confirmed |
| Changed in canonical-devices-system-image: | |
| milestone: | ww02-2016 → ww04-2016 |
| Changed in canonical-devices-system-image: | |
| milestone: | ww04-2016 → ww08-2016 |
| Changed in canonical-devices-system-image: | |
| milestone: | ww08-2016 → 11 |
| status: | In Progress → Fix Committed |
| Changed in canonical-pocket-desktop: | |
| status: | New → Fix Released |
| Changed in qtmir: | |
| status: | In Progress → Fix Committed |
| Changed in qtmir (Ubuntu): | |
| status: | Confirmed → Fix Committed |
| no longer affects: | qtubuntu |
| Changed in unity8 (Ubuntu): | |
| status: | Confirmed → Fix Committed |
| no longer affects: | unity8 |
| no longer affects: | qtubuntu (Ubuntu) |
| Changed in unity8 (Ubuntu): | |
| assignee: | nobody → Daniel d'Andrada (dandrader) |
| Changed in qtmir (Ubuntu): | |
| assignee: | nobody → Daniel d'Andrada (dandrader) |
| Changed in qtmir: | |
| status: | Fix Committed → Fix Released |
| Changed in qtmir (Ubuntu): | |
| status: | Fix Committed → Fix Released |
| Changed in unity8 (Ubuntu): | |
| status: | Fix Committed → Fix Released |
| Changed in ubuntu-ux: | |
| status: | Fix Committed → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | Fix Committed → Fix Released |
| Changed in qtmir (Ubuntu): | |
| importance: | Undecided → Medium |
| no longer affects: | qtmir |
To post a comment you must log in.
Need this one fixed in order to fix
https:/