Unity Control Center

Merge lp://staging/~seb128/unity-control-center/sound-better-icon into lp://staging/unity-control-center

  1. sound-better-icon
  2. Merge into trunk
Proposed by Sebastien Bacher
Status: Merged
Approved by: Iain Lane
Approved revision: 12847
Merged at revision: 12849
Proposed branch: lp://staging/~seb128/unity-control-center/sound-better-icon
Merge into: lp://staging/unity-control-center
Diff against target: 59 lines (+24/-0)
4 files modified
configure.ac (+2/-0)
panels/sound/data/icons/256x256/Makefile.am (+3/-0)
panels/sound/data/icons/256x256/apps/Makefile.am (+18/-0)
panels/sound/data/icons/Makefile.am (+1/-0)
To merge this branch: bzr merge lp://staging/~seb128/unity-control-center/sound-better-icon
Reviewer Review Type Date Requested Status
Iain Lane (community) Approve
Review via email: mp+285452@code.staging.launchpad.net

Commit message

Include a better resolution unity-sound-panel icon

Description of the change

Include a better resolution unity-sound-panel icon

(open the sound panel and alt-tab on unity, see how the current icon is low quality, the update fixes that)

To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote :

I don't get it - the icon you added is actually 48×48

review: Needs Information
lp://staging/~seb128/unity-control-center/sound-better-icon updated
12847. By Sebastien Bacher

with the correct icon it's better

Revision history for this message
Sebastien Bacher (seb128) wrote :

sorry, I tried several variant and back to the old icon to see the difference, looks like I mixed the versions ... updated with the correct one now, thanks for catching the error!

Revision history for this message
Iain Lane (laney) wrote :

what a great looking icon!

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