[frameworks/kirigami] src/controls: Make the GlobalDrawer correctly size to its contents again
Ron El <[email protected]>
| Newsgroups | gmane.comp.kde.cvs |
|---|---|
| Message-ID | <[email protected]> |
Git commit 70bbf12c0fccee5d218f6f559c487e6bf4411a68 by Ron El.
Committed on 05/08/2026 at 13:11.
Pushed by ahiemstra into branch 'master'.
Make the GlobalDrawer correctly size to its contents again
This amends commit 64b13ba1bcbb6ec0d86cd92799ec45661044d46c which may
have been correct about not setting 'width' from the parent, but was
not correct about taking implicitWidth from the layout parent - since
that makes it zero and the drawer collapses to a narrow width eliding
most of its contents.
We need to take the implicitWidth from the content so that it bubbles
up to fit layout width to the Action contents, and since ItemDelegate
(the base of KP.GlobalDrawerActionItem) already has its implicitWidth
set to implicitContentWidth, we can just delete this binding entirely
to get the expected behaviour.
M +0 -1 src/controls/GlobalDrawer.qml
https://invent.kde.org/frameworks/kirigami/-/commit/70bbf12c0fccee5d218f6f559c487e6bf4411a68
diff --git a/src/controls/GlobalDrawer.qml b/src/controls/GlobalDrawer.qml
index 93ee232a7..0016cb746 100644
--- a/src/controls/GlobalDrawer.qml
+++ b/src/controls/GlobalDrawer.qml
@@ -429,7 +429,6 @@ KC.OverlayDrawer {
? Platform.Theme.Window : parent.Platform.Theme.colorSet
visible: !delegate.isExpanded
- implicitWidth: parent.implicitWidth
tAction: delegate.tAction