aacid added a comment.
In D29826#673572 <https://phabricator.kde.org/D29826#673572>, @poboiko wrote: > Thanks for looking into it! :) > > In D29826#673543 <https://phabricator.kde.org/D29826#673543>, @aacid wrote: > > > I don't think moving this code from KIconThemes to kmainwindow makes sense, what about all the apps that use KIconThemes but no KMainWindow? > > > I'm just not aware of any. I've looked at some of the most used (by me :]), noted that all of them use KMainWindow and thought it might be a good option. What about discover for example? REPOSITORY R263 KXmlGui REVISION DETAIL https://phabricator.kde.org/D29826 To: poboiko, aacid, mart, broulik Cc: mart, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns