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