Merge lp:~jjardon/libappindicator/fix-820080 into lp:libappindicator
Proposed by
Javier Jardón
Status: | Superseded | ||||
---|---|---|---|---|---|
Proposed branch: | lp:~jjardon/libappindicator/fix-820080 | ||||
Merge into: | lp:libappindicator | ||||
Diff against target: |
82 lines (+7/-31) 1 file modified
src/app-indicator.c (+7/-31) |
||||
To merge this branch: | bzr merge lp:~jjardon/libappindicator/fix-820080 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ted Gould (community) | Disapprove | ||
Andrea Cimitan (community) | Approve | ||
Review via email: mp+76481@code.launchpad.net |
This proposal has been superseded by a proposal from 2011-09-22.
To post a comment you must log in.
Unmerged revisions
- 215. By Javier Jardón
-
Do not append "-panel" to the given icon name if it's missing.
As the -panel variant can not exist. Also, GtkStatusIcon
support fixed-sized icons nowadays.
I don't know which is the default coding style, but from the snippet I see you might want to remove the space between function names and their brackets