[Differential] [Request, 78 lines] D2281: Prepare for retiring the C++ text label in Task Manager delegates.

[hein (Eike Hein)]

hein created this revision.
hein added reviewers: Plasma, mart.
hein added a subscriber: plasma-devel.
Restricted Application added a project: Plasma.

REVISION SUMMARY
  The Task Manager button delegates currently use a custom C++
  QQuickPaintedItem to display text. This is undesirable because
  it causes a lot of texture traffic and bypasses the glyph cache
  of Qt Quick's OpenGL renderer. However, the C++ code does
  complex things - like per-line fadeouts and shadowing - that are
  not easily recreated in pure QML, which has kept the C++ label
  around up until now`.
  
  This patch gets us on the way by putting the existing label
  implementation behind a Loader, and making a pure-QML label
  implementation available via an experimental config option.
  
  I'd like to have this in the 5.7 branch because one of our
  downstreams (Netrunner Linux) used to ship similar code in
  a fork of the applet for visual reasons, and since what they
  want to ship aligns with our long-term plans, this will allow
  them to avoid the fork while getting us an early tester for
  upstream code.
  
  As a nice bonus it ends up being a performance optimization
  for Icontasks users, as TextLabel will no longer even be
  instanciated there.

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

BRANCH
  Plasma/5.7

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

AFFECTED FILES
  applets/taskmanager/package/contents/config/main.xml
  applets/taskmanager/package/contents/ui/CppTextLabel.qml
  applets/taskmanager/package/contents/ui/QmlTextLabel.qml
  applets/taskmanager/package/contents/ui/Task.qml

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: hein, #plasma, mart
Cc: plasma-devel, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160725/9a6c0d41/attachment.html>