[Differential] [Commented On] D2418: Drop the C++ text label implementation.

hein (Eike Hein) noreply at phabricator.kde.org
Fri Aug 12 09:12:15 UTC 2016


hein added inline comments.

INLINE COMMENTS

> broulik wrote in Task.qml:390
> Should we perhaps keep the Loader for optimizing the icon-tasks case, or does that introduce too much overhead/code maintenance?

I think it's a wash. If we add the Loader, we have an extra QObject, an extra QML file, it adds instanciation latency and needs more disk I/O and parsing work. An invisible label isn't that heavy I think :).

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

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

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

To: hein, #plasma, broulik
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160812/b5da1cc8/attachment.html>


More information about the Plasma-devel mailing list