Merge lp://staging/~mvo/libappindicator/fix-820080 into lp://staging/libappindicator
Proposed by
Michael Vogt
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Ted Gould | ||||
Approved revision: | 226 | ||||
Merged at revision: | 221 | ||||
Proposed branch: | lp://staging/~mvo/libappindicator/fix-820080 | ||||
Merge into: | lp://staging/libappindicator | ||||
Diff against target: |
63 lines (+39/-7) 1 file modified
src/app-indicator.c (+39/-7) |
||||
To merge this branch: | bzr merge lp://staging/~mvo/libappindicator/fix-820080 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ted Gould (community) | Approve | ||
Michael Vogt (community) | Needs Resubmitting | ||
Review via email: mp+77676@code.staging.launchpad.net |
Description of the change
This checks for the long item name before using it to ensure that apps that do not provide a "-panel" fallback icon still work.
This probably needs a additional check for "icon_theme_path" to ensure its catching custom icon theme paths, I'm happy to add this once I get approval for the general approach.
(this is against the right branch this time)
To post a comment you must log in.
Tested in several configurations: XFce, GNOME Classic, Ubuntu and seems to work ok.