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

Reply via email to