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