[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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.