diff options
| author | Sebastian Englbrecht <[email protected]> | 2026-05-23 17:40:53 +0200 |
|---|---|---|
| committer | Méven Car <[email protected]> | 2026-05-28 08:59:00 +0000 |
| commit | 680ccee08cfac2b53bb7bf7d35b780889efa4b51 (patch) | |
| tree | d9c3bb191c4353a577bd11c4f9bd1b5297c12885 | |
| parent | 714830f8dda873fbefa6de9704c90220e16a6133 (diff) | |
dolphinmainwindow,kitemlistview: fix unowned object allocations
Parent QMenu to the main window so it is owned and deleted with it.
Register KItemListContainerAccessible in Qt's accessibility cache so
Qt takes ownership and deletes it with the container widget.
| -rw-r--r-- | src/dolphinmainwindow.cpp | 2 | ||||
| -rw-r--r-- | src/kitemviews/kitemlistview.cpp | 2 |
2 files changed, 3 insertions, 1 deletions
diff --git a/src/dolphinmainwindow.cpp b/src/dolphinmainwindow.cpp index 7d798ac17..ebd811efa 100644 --- a/src/dolphinmainwindow.cpp +++ b/src/dolphinmainwindow.cpp @@ -2875,7 +2875,7 @@ void DolphinMainWindow::updateSplitActions() popoutSplitAction->setEnabled(true); if (!m_splitViewAction->menu()) { setActionPopupMode(m_splitViewAction, QToolButton::MenuButtonPopup); - m_splitViewAction->setMenu(new QMenu); + m_splitViewAction->setMenu(new QMenu(this)); m_splitViewAction->addAction(popoutSplitAction); } } else { diff --git a/src/kitemviews/kitemlistview.cpp b/src/kitemviews/kitemlistview.cpp index 452567f05..b87216d4b 100644 --- a/src/kitemviews/kitemlistview.cpp +++ b/src/kitemviews/kitemlistview.cpp @@ -353,6 +353,8 @@ void KItemListView::setAccessibleParentsObject(KItemListContainer *accessiblePar { Q_ASSERT(!m_accessibleParent); m_accessibleParent = new KItemListContainerAccessible(accessibleParentsObject); + // Make sure Qt's accessibility cache takes ownership; tests have no AT client. + QAccessible::queryAccessibleInterface(accessibleParentsObject); } KItemListContainerAccessible *KItemListView::accessibleParent() { |