D27182: [applets/taskmanager] Clean up Tooltip code

Nathaniel Graham noreply at phabricator.kde.org
Thu Feb 6 05:01:48 GMT 2020


ngraham created this revision.
ngraham added reviewers: Plasma, VDG.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ngraham requested review of this revision.

REVISION SUMMARY
  The ToolTip code was kind of messy. This patch adjusts it to use modern Layouts,
  fixes some binding loops, reduces redundancy, adds comments, and ports almost
  everything to PlasmaComponents3. We still need PC2 for `Highlight` so it can't
  be removed entirely.
  
  There is only one unavoidable visual change, which was made in this patch
  because it is consistent with the latest VDG mockup (T12640 <https://phabricator.kde.org/T12640>): the blurred
  album art background no longer goes under the player controls bar. This
  simplifies the code substantially, and as the mockup shows, it just looks better
  anyway (IMO).
  
  This is not intended to fully implement the mockup in T12640 <https://phabricator.kde.org/T12640>.

TEST PLAN
  Everything looks the same, except for this:
  
  F8087660: Peek 2020-02-05 21-49.webm <https://phabricator.kde.org/F8087660>

REPOSITORY
  R119 Plasma Desktop

BRANCH
  clean-up-tooltip-code (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27182

AFFECTED FILES
  applets/taskmanager/package/contents/ui/ToolTipInstance.qml

To: ngraham, #plasma, #vdg
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200206/c9c4301d/attachment.html>


More information about the Plasma-devel mailing list