D26323: [KCM/Component] Terminal port to KConfigXT, make isDefault work
Méven Car
noreply at phabricator.kde.org
Tue Dec 31 10:45:55 GMT 2019
meven marked an inline comment as done.
meven added a comment.
In D26323#585271 <https://phabricator.kde.org/D26323#585271>, @ervin wrote:
> One nitpick to address before pushing.
>
> Also I mark those accepted assuming there will be another phase where the settings objects will actually be properly matched to the GUI. Currently it's all fixing the isDefaults by hand based on GUI state but not on config state.
Given the current GUI entanglement and the CfpPlugin split, I wonder about the opportunity given how much rewrite/refactoring will be necessary to use the KConfig state.
> I don't think it currently honors the mutability state of those keys. This will need being addressed in further commits.
Indeed, it will follow.
REPOSITORY
R119 Plasma Desktop
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D26323
To: meven, ervin, #plasma
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/20191231/d71aa4c1/attachment-0001.html>
More information about the Plasma-devel
mailing list