The Sound Menu

Merge lp://staging/~ted/indicator-sound/greeter-no-settings into lp://staging/indicator-sound/13.10

  1. greeter-no-settings
  2. Merge into trunk.13.10
Proposed by Ted Gould
Status: Merged
Approved by: Ted Gould
Approved revision: 389
Merged at revision: 387
Proposed branch: lp://staging/~ted/indicator-sound/greeter-no-settings
Merge into: lp://staging/indicator-sound/13.10
Diff against target: 68 lines (+14/-4)
3 files modified
data/com.canonical.indicator.sound (+1/-1)
src/service.vala (+1/-0)
src/sound-menu.vala (+12/-3)
To merge this branch: bzr merge lp://staging/~ted/indicator-sound/greeter-no-settings
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Lars Karlitski (community) Approve
Review via email: mp+188911@code.staging.launchpad.net

Commit message

Add a desktop_greeter menu without settings

Description of the change

Adds a greeter menu and makes it so we can configure whether menus have settings or not.

To post a comment you must log in.
lp://staging/~ted/indicator-sound/greeter-no-settings updated
389. By Ted Gould

Make sure to set the initial value of settings shown

Revision history for this message
Lars Karlitski (larsu) wrote :

That should do it. Thanks!

review: Approve
Revision history for this message
Ted Gould (ted) wrote :

Mterry verified it worked for his use-case in the greeter.

Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

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