[Differential] [Changed Subscribers] D2613: Fix some status notifier items not appearing

broulik (Kai Uwe Broulik) noreply at phabricator.kde.org
Tue Aug 30 12:51:01 UTC 2016


broulik added inline comments.

INLINE COMMENTS

> statusnotifieritemsource.cpp:85
> +    //when it gets data with a new key it tries to update the  QAIM roleNames
> +    //from QML this achieves absolutely nothing as there is no signal to tell QQmlDelegateModel to reload the roleNames in QQmlAdapatorModel
> +    //no matter if the row changes or the model refreshes

Wrong indentation

> statusnotifieritemsource.cpp:90
> +    //by setting everything up-front so that we have all role names when we call the first checkForUpdate()
> +    setData(QStringLiteral("AttentionIcon"), QString());
> +    setData(QStringLiteral("AttentionIconName"), QString());

AttentionIcon is a QIcon

> statusnotifieritemsource.cpp:94
> +    setData(QStringLiteral("Category"), QString());
> +    setData(QStringLiteral("Icon"), QString());
> +    setData(QStringLiteral("IconName"), QString());

This is also a QIcon

> statusnotifieritemsource.cpp:106
> +    setData(QStringLiteral("ToolTipChanged"), false);
> +    setData(QStringLiteral("ToolTipIcon"), QString());
> +    setData(QStringLiteral("ToolTipSubTitle"), QString());

This apparently can be both QString and QIcon...

> statusnotifieritemsource.cpp:111
> +
> +
> +

Two superfluous empty lines

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

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

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


More information about the Plasma-devel mailing list