https://bugs.kde.org/show_bug.cgi?id=480173
Marco Martin <notm...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Status|ASSIGNED |RESOLVED
Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
|ma/plasma-workspace/-/commi |ma/plasma-workspace/-/commi
|t/63e572e923a4684edb7e90c5e |t/9c872a54287a1ea84326a7100
|a7eb02d967b39fb |1386e4040b7c5a4
Version Fixed In| |6.2
--- Comment #17 from Marco Martin <notm...@gmail.com> ---
Git commit 9c872a54287a1ea84326a71001386e4040b7c5a4 by Marco Martin.
Committed on 27/06/2024 at 07:50.
Pushed by mart into branch 'master'.
applets/systemtray: Fix shortcut activation for hidden applets
The systray applet was opened by a connection to activated in the
PlasmoidItem container element. This element is a delegate, of a Repeater
for active applets, and for the internal GridView in the popup for the passive
applets.
Being a Delegate we can't be guaranteed it will be instantiated at startup,
in particular it won't be unless the first time the gridview is visible,
so the connections to the activated signal won't work until the first time the
applet is open. Also, if there would ever be enough icons in the popup to
cause a scrollbar, it will destroy the PopupItems out of view.
Move the connection to the activate signal that will cause the popup to open
inside an Instantiator in the main qml file: with an Instantiator we will
always
be guaranteed every delegate (which is just a Connections)
will always be instantiated all the time
Depends on https://invent.kde.org/plasma/libplasma/-/merge_requests/1145
Related: bug 488619
FIXED-IN: 6.2
M +1 -8 applets/systemtray/package/contents/ui/HiddenItemsView.qml
M +0 -1 applets/systemtray/package/contents/ui/items/PlasmoidItem.qml
M +59 -8 applets/systemtray/package/contents/ui/main.qml
https://invent.kde.org/plasma/plasma-workspace/-/commit/9c872a54287a1ea84326a71001386e4040b7c5a4
--
You are receiving this mail because:
You are watching all bug changes.
