David Faure wrote: > On Thursday 08 March 2012 13:11:37 Giorgos Tsiapaliwkas wrote: >> This is my patch about moving kdialog to staging/kwidgets but I need to >> link to >> KGuiItem(KDialog requires that). I can do then job, but there was nobody >> in the channel to approve it. So I didn't. > > This KDialog move looks good. > > KGuiItem should move to kguiaddons I think. > Which means moving KIcon and KIconLoader/KIconTheme too (KGuiItem depends > on it). > > Right Steve? We said we would keep KIcon/KIconLoader --> kguiaddons ? >
Well, considering that QIcon is in QWidgets, there's a widgets dependency anyway. Is KGuiAddons supposed to be widgets-free? We did say we'd keep the KIcon{Loader,Engine,Theme} stuff, yes. Whether we deprecate the KIcon class or replace it with methods in a namespace, I'm not sure. Thanks, Steve. _______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel