From 9add143a40f10a85fb5d9c319013381c1458ba02 Mon Sep 17 00:00:00 2001 From: Ismael Asensio Date: Fri, 18 Aug 2023 17:06:11 +0200 Subject: [PATCH] kcms/decoration: Fix highlight when hovering the preview The preview item implements hover event filters, which by default are accepted and not propagated to their parent, breaking the hightlight on hover for the GridView. We can simply "ignore()" those events so they are propagated, but still fully functional. Preemptively, do the same also for mouse events, which seem to be already propagated now, but shouldn't according to the QQuickItem documentation. CCBUG: 473011 --- src/kcms/decoration/declarative-plugin/previewitem.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/kcms/decoration/declarative-plugin/previewitem.cpp b/src/kcms/decoration/declarative-plugin/previewitem.cpp index ea6df5d8c3..2fb4817bd9 100644 --- a/src/kcms/decoration/declarative-plugin/previewitem.cpp +++ b/src/kcms/decoration/declarative-plugin/previewitem.cpp @@ -298,6 +298,8 @@ void PreviewItem::proxyPassEvent(E *event) const } else { QCoreApplication::instance()->sendEvent(decoration(), event); } + // Propagate events to parent + event->ignore(); } void PreviewItem::mouseDoubleClickEvent(QMouseEvent *event)