https://bugs.kde.org/show_bug.cgi?id=420993

Konrad Materka <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/plasma-workspace/commit/
                   |                            |2ce60f4e502a29e7ce468fc9796
                   |                            |fa5088a3486f6
             Status|CONFIRMED                   |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Konrad Materka <[email protected]> ---
Git commit 2ce60f4e502a29e7ce468fc9796fa5088a3486f6 by Konrad Materka.
Committed on 25/05/2020 at 22:17.
Pushed by kmaterka into branch 'Plasma/5.19'.

[applets/systemtray] Lazy initialize SystemTray model

Summary:
SystemTray model was created in constructor, delay initialization and
create it lazily.
The `SystemTray::appletAdded` signal is connected to both SystemTray's
model and `Container`. `Container` must be connected first, because it sets
some properties of newly added applet: `visible` to false and `parent`
to the root item (container). If `Container` is connected second, then
newly created applet ios added to the model and rendered, but immediately
after that `Container` changes visibility to false.

Test Plan:
1. Do not play anything (no Medial Player icon at all)
2. Run Gwenview
3. Check if Medial Player icon in hidden view is rendered correct.
Before change:
Sometimes icon is not rendered, looks like invisible or fully transparent.
After change:
Icon rendered correctly.

M  +27   -15   applets/systemtray/systemtray.cpp
M  +4    -2    applets/systemtray/systemtray.h

https://invent.kde.org/plasma/plasma-workspace/commit/2ce60f4e502a29e7ce468fc9796fa5088a3486f6

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to