Merge lp://staging/~larryprice/ubuntu-app-launch/better-icon-themes into lp://staging/ubuntu-app-launch

Proposed by Larry Price
Status: Merged
Approved by: Michael Terry
Approved revision: 298
Merged at revision: 298
Proposed branch: lp://staging/~larryprice/ubuntu-app-launch/better-icon-themes
Merge into: lp://staging/ubuntu-app-launch
Diff against target: 156 lines (+62/-39)
1 file modified
libubuntu-app-launch/application-icon-finder.cpp (+62/-39)
To merge this branch: bzr merge lp://staging/~larryprice/ubuntu-app-launch/better-icon-themes
Reviewer Review Type Date Requested Status
Michael Terry Approve
Review via email: mp+319358@code.staging.launchpad.net

Commit message

Add all icon directories to search paths regardless of context.

Description of the change

Add all icon directories to search paths regardless of context. Also add a couple more default themes, and fix the "manual" search method.

To post a comment you must log in.
296. By Larry Price

gnome

297. By Larry Price

format

298. By Larry Price

reorder themes and add suru

Revision history for this message
Michael Terry (mterry) wrote :

Looks good to me. I tested it and it now shows an icon for all apps that declare one! Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches