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

Revision history for this message
Antti Kaijanmäki (kaijanmaki) wrote :

Seems I hit a couple of bugs:

notify-osd and notify-osd-icons packages are not providing index.theme :/
QIcon::themedIcon is not looking up all the base directories. It just takes the first one with matching theme directory and gives up if the requested icon is not found. Thus notify-osd icons are not found (after adding the index.theme).

Both needs fixing.

« Back to merge proposal