Bug 375112 - Touchpad indicator moves itself in system tray when clicked
Summary: Touchpad indicator moves itself in system tray when clicked
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: System Tray (show other bugs)
Version: 5.8.95
Platform: Arch Linux Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-01-15 20:51 UTC by Patrick Silva
Modified: 2017-02-08 16:28 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.10.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2017-01-15 20:51:14 UTC
Enable the touchpad indicator in system tray settings and click on its icon.
Watch the video please.
I'm running plasma 5.9 beta on Arch.

https://www.youtube.com/watch?v=Gql9Y-_2yxk
Comment 1 Nikolaos Kakouros 2017-02-07 20:30:14 UTC
This bug looks related to bug 365630.
Comment 2 Kai Uwe Broulik 2017-02-07 21:48:51 UTC
Somewhat related to Bug 365630 but the touchpad KCM switches from ActiveStatus to AcceptingInputStatus which causes System Tray to re-assess its visibility and then moves it to the front, although its effective visibility didn't actually change.
Comment 3 Kai Uwe Broulik 2017-02-08 16:28:27 UTC
Git commit a7d56be57a1c1258608d8eb363191d69f40c12e3 by Kai Uwe Broulik.
Committed on 08/02/2017 at 16:26.
Pushed by broulik into branch 'master'.

[System Tray] Introduce "effectiveStatus" property and update visibility only when that changes

Instead of calling it in response to any of the properties it depends on.

This also saves some calls to this function on startup (80 instead of 90 for me with 14 tray icons)
and keeps plasmoids from shifting around just because their status changed from e.g. Active
to AcceptingInput (like touchpad when it asks for whether you really want to disable touchpad
with no mouse attached) although the item's effective visibility didn't actually change.
Related: bug 365630
FIXED-IN: 5.10.0

Differential Revision: https://phabricator.kde.org/D4488

M  +10   -4    applets/systemtray/package/contents/ui/items/AbstractItem.qml
M  +7    -8    applets/systemtray/package/contents/ui/main.qml

https://commits.kde.org/plasma-workspace/a7d56be57a1c1258608d8eb363191d69f40c12e3