D14391: Fix random order of plugin's config pages
Gleb Popov
noreply at phabricator.kde.org
Sun Jan 27 17:55:31 GMT 2019
arrowd added inline comments.
INLINE COMMENTS
> mwolff wrote in uicontroller.cpp:542
> so the source of randomness can only come from here, where we iterate over the loaded plugins. fixing that would be simple, too: just collect the pages here, then sort them, then add them. that should fix the randomness without changing the whole API and changing the order of the basic config pages
This approach would sort pages when the configuration window is opened. It doesn't sort pages when you load/unload plugins from `Plugins` page.
When I uncheck and then check `Clazy` plugin there, it gets appended to the end of `Analyzers` pages list. But if I then close and open the configuration window again, it gets back on its place.
If you are OK with this, so do I.
REPOSITORY
R32 KDevelop
REVISION DETAIL
https://phabricator.kde.org/D14391
To: arrowd, #kdevelop, antonanikin, mwolff
Cc: mwolff, arrowd, apol, kossebau, kdevelop-devel, glebaccon, hase, antismap, iodelay, geetamc, Pilzschaf, akshaydeo, surgenight
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20190127/d5492767/attachment.html>
More information about the KDevelop-devel
mailing list