ngraham created this revision. ngraham added reviewers: VDG, Plasma. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ngraham requested review of this revision.
REVISION SUMMARY As discussed in T12778 <https://phabricator.kde.org/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. 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 F8150019: Widget.png <https://phabricator.kde.org/F8150019> F8150017: Systray item.png <https://phabricator.kde.org/F8150017> F8150021: Icon-less tooltips.webm <https://phabricator.kde.org/F8150021> REPOSITORY R242 Plasma Framework (Library) BRANCH remove-tooltip-icon (branched from master) REVISION DETAIL https://phabricator.kde.org/D27848 AFFECTED FILES src/declarativeimports/core/private/DefaultToolTip.qml To: ngraham, #vdg, #plasma Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns