D20810: Fix launcher URL comparison
Kai Uwe Broulik
noreply at phabricator.kde.org
Thu Apr 25 14:19:24 BST 2019
broulik created this revision.
broulik added reviewers: Plasma, hein, ngraham, mlaurent.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.
REVISION SUMMARY
Launcher URL is a `QUrl` which doesn't strictly map empty `string`.
TEST PLAN
Pin to task bar and start new instance is correctly disabled now if no launcher url could be determined
Yet another regression caused by 6e36de2e1e92743751ddd5b735cdfe9b2e95e5e2 <https://phabricator.kde.org/R119:6e36de2e1e92743751ddd5b735cdfe9b2e95e5e2>
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D20810
AFFECTED FILES
applets/taskmanager/package/contents/ui/ContextMenu.qml
To: broulik, #plasma, hein, ngraham, mlaurent
Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190425/76ee82f8/attachment-0001.html>
More information about the Plasma-devel
mailing list