Merge lp://staging/~larsu/indicator-messages/lp956147 into lp://staging/indicator-messages/0.3

Proposed by Lars Karlitski
Status: Merged
Approved by: Charles Kerr
Approved revision: 264
Merged at revision: 265
Proposed branch: lp://staging/~larsu/indicator-messages/lp956147
Merge into: lp://staging/indicator-messages/0.3
Diff against target: 21 lines (+3/-0)
1 file modified
src/messages-service.c (+3/-0)
To merge this branch: bzr merge lp://staging/~larsu/indicator-messages/lp956147
Reviewer Review Type Date Requested Status
Charles Kerr (community) Approve
Review via email: mp+100836@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Charles Kerr (charlesk) wrote :

I agree with you that this isn't a perfect patch, but doing things right would take more time than we have right now. It's better to put in this quick fix than to ship with the "broken" icons.

Considering the tradeoffs, this is the right fix for 12.04.

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