[applet/taskmanager] Remove the launcher items' tooltip icons
ClosedPublic

Authored by ngraham on Mar 4 2020, 9:58 PM.

Details

Summary

As discussed in T12778, the icon can never be in a correct state: either it's
redundant with the icon you're hovering over that made the tooltip appear, or
else it's different from the icon you're hovering over, which is inconsistent.
Furthermore, some instances of the tooltip have icons and others do not,
heightening the inconsistency.

Since either case is undesirable, let's just remove the icon from the tooltip
entirely. This reduces the visual noise in the tooltip and solves the entire
class of bugs automatically.

Test Plan


Diff Detail

Repository
R119 Plasma Desktop
Branch
remove-tooltip-icon (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 23273
Build 23291: arc lint + arc unit
ngraham created this revision.Mar 4 2020, 9:58 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMar 4 2020, 9:58 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham requested review of this revision.Mar 4 2020, 9:58 PM
niccolove accepted this revision as: niccolove.Mar 4 2020, 9:59 PM
This revision is now accepted and ready to land.Mar 4 2020, 9:59 PM
ognarb accepted this revision.Mar 5 2020, 1:16 AM

Why?

I think you forgot to read the description section. :)

This revision was automatically updated to reflect the committed changes.