Any news about this KIconEngine question?

David Faure faure at kde.org
Fri Sep 28 19:13:56 UTC 2012


Hi Antonio,

Did you get a chance to find out more about this code from KIconEngine?


    if (QWidget* targetWidget = dynamic_cast<QWidget*>(painter->device())) {
        if (qobject_cast<QMenu*>(targetWidget))
            group = KIconLoader::Small;
        else if (qobject_cast<QToolBar*>(targetWidget->parent()))
            group = KIconLoader::Toolbar;
    }

I'd like to get rid of that hack so that the iconloader doesn't depend on QtWidgets,
but I'm not sure what this really does.

Maybe it's for pure QMenus/QToolbars, given that KMenus and KToolbars already request
the right 'group'? Or maybe they don't, that's the whole question :)

-- 
David Faure, faure at kde.org, http://www.davidfaure.fr
Working on KDE, in particular KDE Frameworks 5



More information about the Kde-frameworks-devel mailing list