D26391: DesktopPaths KCM: Remove the moving directory logic
Kevin Ottens
noreply at phabricator.kde.org
Fri Jan 3 11:55:41 GMT 2020
ervin created this revision.
ervin added reviewers: Plasma, crossi, bport, meven, mart, davidedmundson.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ervin requested review of this revision.
REVISION SUMMARY
This simplifies greatly this otherwise mundane KCM. It introduced a lot
of complexity while gettings things wrong anyway: it would miss if a
directory is included in another one except in the "autostart inside
desktop" case, it would also leave you stranded in case of an
uncompleted move.
So now the KCM has a clear purpose: edit the settings. Moving data
around is still up to the user using a proper filemanager.
Long term it would probably be better for the filemanager to edit the
settings automatically when the user move on of those folders around,
then the KCM could be decommissioned.
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D26391
AFFECTED FILES
kcms/desktoppaths/globalpaths.cpp
kcms/desktoppaths/globalpaths.h
To: ervin, #plasma, crossi, bport, meven, mart, davidedmundson
Cc: plasma-devel, 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/20200103/c516789c/attachment.html>
More information about the Plasma-devel
mailing list