The Sound Menu

Merge lp://staging/~themuso/indicator-sound/accessible-desc into lp://staging/~indicator-applet-developers/indicator-sound/trunk_3

accessible-desc
Merge into trunk_3

Proposed by Conor Curran on 2011-02-16

Status:	Merged
Merged at revision:	204
Proposed branch:	lp://staging/~themuso/indicator-sound/accessible-desc
Merge into:	lp://staging/~indicator-applet-developers/indicator-sound/trunk_3
Diff against target:	144 lines (+41/-4) 4 files modified po/POTFILES.in (+1/-0) src/indicator-sound.c (+17/-1) src/volume-widget.c (+21/-2) src/volume-widget.h (+2/-1)
To merge this branch:	bzr merge lp://staging/~themuso/indicator-sound/accessible-desc
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Conor Curran (community)			Approve on 2011-02-24
Review via email: mp+49983@code.staging.launchpad.net

Description of the change

Just to keep this in view, resubmit merge proposal

lp://staging/~themuso/indicator-sound/accessible-desc updated on 2011-02-18

202. By Conor Curran on 2011-02-18: voip feature landed
203. By Conor Curran on 2011-02-18: bumped for new release

Revision history for this message

Conor Curran (cjcurran) wrote on 2011-02-21:

I don't know why this is but half of this merge request (the previous work) has been omitted. As you can see below its only prepared the merge the memory leak fix. Most of the a11y stuff has been left out.

review: Needs Fixing

Revision history for this message

Luke Yelavich (themuso) wrote on 2011-02-21:

On Mon, Feb 21, 2011 at 09:59:01PM EST, Conor Curran wrote:
> Review: Needs Fixing
> I don't know why this is but half of this merge request (the previous work) has been omitted. As you can see below its only prepared the merge the memory leak fix. Most of the a11y stuff has been left out.

Ok will take everything I've done and rebase on top of current trunk and re-submit.

Luke

lp://staging/~themuso/indicator-sound/accessible-desc updated on 2011-02-22