D27109: Fix duplicate results for sessions, simplify and optimize

Alexander Lohnau noreply at phabricator.kde.org
Sun Feb 2 19:37:30 GMT 2020


alex created this revision.
alex added reviewers: broulik, davidedmundson, ngraham.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
alex requested review of this revision.

REVISION SUMMARY
  The sessions are now only loaded when the plugin is initialized or the folder changes.
  Additionally the option to search for a session just by the name has been removed,
  because Kate itself provides this functionality (demonstated in screenshots).

TEST PLAN
  Create sessions, they should show up like in the last two screenshots.
  After testing this create/edit/delete sessions, the changes should be automatically applied.
  Before:
  F8078772: before_duplicate_launch_entries.png <https://phabricator.kde.org/F8078772>
  After:
  F8078770: after_no_duplicates.png <https://phabricator.kde.org/F8078770>
  Search with "kate" prefix should work as before:
  F8078771: works_same.png <https://phabricator.kde.org/F8078771>

REPOSITORY
  R114 Plasma Addons

BRANCH
  katesession_improvements (branched from master)

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

AFFECTED FILES
  runners/katesessions/CMakeLists.txt
  runners/katesessions/katesessions.cpp
  runners/katesessions/katesessions.h

To: alex, broulik, davidedmundson, ngraham
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200202/00f35f06/attachment.html>


More information about the Plasma-devel mailing list