Merge lp://staging/~charlesk/gnome-control-center/indicator-visibility into lp://staging/~ubuntu-desktop/gnome-control-center/ubuntu

Proposed by Charles Kerr
Status: Work in progress
Proposed branch: lp://staging/~charlesk/gnome-control-center/indicator-visibility
Merge into: lp://staging/~ubuntu-desktop/gnome-control-center/ubuntu
Diff against target: 58 lines (+43/-0)
2 files modified
debian/patches/60_add_sound_indicator_visibility.patch (+42/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp://staging/~charlesk/gnome-control-center/indicator-visibility
Reviewer Review Type Date Requested Status
Conor Curran (community) Approve
Ubuntu Desktop Pending
Review via email: mp+90930@code.staging.launchpad.net

Description of the change

Adds a visibility toggle for the sound indicator for Bug #829648

This is my first submission of a downstream patch for an upstream package, so cluestick lessons are welcomed.

To post a comment you must log in.
Revision history for this message
Charles Kerr (charlesk) wrote :

Switching to "Work in progress" -- this relies on indicator-sound 0.8.1.0-plus-one to be released first

334. By Charles Kerr

Use g_settings_bind() instead of reinventing the wheel with a bunch of GSettings<->checkbox glue code

Revision history for this message
Conor Curran (cjcurran) :
review: Approve
Revision history for this message
Robert Ancell (robert-ancell) wrote :

Is this still applicable?

Unmerged revisions

334. By Charles Kerr

Use g_settings_bind() instead of reinventing the wheel with a bunch of GSettings<->checkbox glue code

333. By Charles Kerr

 * 60_add_sound_indicator_visibility.patch
   - New patch to add a visibility toggle for the sound indicator (LP: #829648)

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