D22306: Do not skip code launching application in application dashboard

Nathaniel Graham noreply at phabricator.kde.org
Sun Jul 7 19:50:02 BST 2019


ngraham added a reviewer: Plasma.
ngraham accepted this revision.
ngraham added a subscriber: hein.
ngraham added a comment.
This revision is now accepted and ready to land.


  I tested this with both mouse and touch. Though I could not reproduce the original issue, this patch does not introduce any regressions for me; in all cases, clicking/tapping and dragging both work correctly 100% of the time. The code change looks sensible as well.
  
  I'd like at least one #plasma <https://phabricator.kde.org/tag/plasma/> review before we land this. Preferably @hein since he's most familiar with the code, but IIRC he's on vacation right now so I think anyone else would be fine too.

REPOSITORY
  R119 Plasma Desktop

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

To: luc4, ngraham, trmdi, #plasma
Cc: hein, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190707/96e4b57c/attachment.html>


More information about the Plasma-devel mailing list