[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