Code review comment for lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/unity-theme-icon-provider

Revision history for this message
Michal Hruby (mhr3) wrote :

135 + QIcon::setThemeSearchPaths(paths);

Unnecessary.

69 + engine->addImageProvider(QLatin1String("theme"), new UnityThemeIconProvider);

The same provider is registered in libqmenumodel, is that one going to be removed?

Would be nice if someone could fix the conflict too.

review: Needs Information

« Back to merge proposal