Improve Noise's sound menu integration

Bug #995060 reported by Mario Guerriero
22
This bug affects 5 people
Affects Status Importance Assigned to Milestone
elementary OS
Fix Released
Undecided
Victor Martinez

Bug Description

Noise should have its entry in the Music Menu

Related branches

Revision history for this message
Eduard Gotwig (gotwig) wrote :

It has its entry in the Music Menu, if you start it once. More information needed.

Changed in noise:
status: New → Incomplete
Revision history for this message
Mario Guerriero (mefrio-g) wrote :

No other information. If I open the Music Menu I can see Rhythmbox, Banshee but not Noise, that's all

Changed in noise:
status: Incomplete → New
Revision history for this message
Brendan William (bwilliam) wrote :

oneiric?

Revision history for this message
Victor Martinez (victored) wrote :

You have to run noise at least once for it to apear in the sound menu, although I agree that it should be there from the start. In that case you may want to target the bug at elementaryOS. I wonder what Ubuntu does with Rhythmbox (I hope they're not hardcoding it inside indicator-sound-menu.)

Changed in noise:
status: New → Invalid
Revision history for this message
Mario Guerriero (mefrio-g) wrote :

I use Noise as my default player so I run it every time I use the pc but it is probably only a bug of my machine

Revision history for this message
Victor Martinez (victored) wrote :

If that's the case I'll mark it as incomplete until we figure out what's wrong. If you're using Oneiric, it's a known bug and we shouldn't be doing anything about it.

Changed in noise:
status: Invalid → Incomplete
Revision history for this message
Mario Guerriero (mefrio-g) wrote : Re: [Bug 995060] Re: Music Menu support

No I use Precise

Mario Guerriero
Sent from iPhone 3GS

Il giorno 05/mag/2012, alle ore 23:41, Victor Eduardo <email address hidden> ha scritto:

> If that's the case I'll mark it as incomplete until we figure out what's
> wrong. If you're using Oneiric, it's a known bug and we shouldn't be
> doing anything about it.
>
> ** Changed in: noise
> Status: Invalid => Incomplete
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/995060
>
> Title:
> Music Menu support
>
> Status in Noise Audio Player:
> Incomplete
>
> Bug description:
> Noise should have its entry in the Music Menu
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/noise/+bug/995060/+subscriptions

Revision history for this message
Victor Martinez (victored) wrote : Re: Music Menu support

I investigated more about this issue and found that Rhythmbox's desktop-file name is hardcoded in indicator-sound, and that's why it appears to be always active. It is also present in the GSettings schema by default.

I've started working on that indicator to achieve the same with Noise. It works. I'm pushing it to lp:~victored/+junk/elementary-sound-menu

affects: noise → elementaryos
Changed in elementaryos:
status: Incomplete → Confirmed
summary: - Music Menu support
+ Improve Noise's sound menu integration
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

And I can alter GSettings defaults if needed.

Revision history for this message
Victor Martinez (victored) wrote :

Sergey: I'll do, thanks.

Changed in elementaryos:
assignee: nobody → Victor Eduardo (victored)
status: Confirmed → In Progress
Revision history for this message
Victor Martinez (victored) wrote :

A fix has been committed to lp:~victored/elementaryos/indicator-sound

Commit message:

    Allow setting preferred media players through a settings key instead of hardcoding them.

    Originally this was done by hardcoding "rhythmbox.desktop" in PlayerController.vala. Now it's possible to set the list of preferred players by modifying the "preferred-media-players" key located at com.canonical.indicator.sound.* Modifications to that key are applied at run-time and it's always monitored. Of course this setting key only applies to non-blacklisted players.

    Currently, setting a player as "preferred" only makes its controls visible all the time.

    Also added BeatBox and Noise to the "interested-media-players" and "preferred-media-players" keys, and removed "rhythmbox" from the latter.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Victor, would you propose your fixes upstream? That seems to be an upstreamable change and that would free you from the burden of maintaining the patch.

Revision history for this message
Victor Martinez (victored) wrote :

Yes, I was thinking the same. I will make a separate branch from the last revision and propose it upstream.

Revision history for this message
Victor Martinez (victored) wrote :
Cody Garver (codygarver)
Changed in elementaryos:
milestone: none → luna-beta1
Cody Garver (codygarver)
Changed in elementaryos:
status: In Progress → Fix Committed
Revision history for this message
Pranav (pranavkaranjkar) wrote :

 I am having trouble with its music menu integration in the daily build of luna. Noise shows up in the music menu and until few days ago, everything was working as it was supposed to, but now out of sudden the musicmenu controls for noise doesn't seem to do anything..

Revision history for this message
Pranav (pranavkaranjkar) wrote :

Also stopped getting notifications for noise.. Is there a way I can restore all the noise settings systemwide to see if it is a problem with my installation?

Changed in elementaryos:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.