[feisty] stop hotkey not working

Bug #99817 reported by Nicolò Chieffo
0
Affects Status Importance Assigned to Milestone
Banshee
Fix Released
Low
banshee (Ubuntu)
Fix Released
Undecided
Chris Halse Rogers

Bug Description

Binary package hint: banshee

I know that the gnome policy is to make users to forget the existance of the possibility to stop a song.
They now can only pause a song. This is ok for me.

Please implement this handler for the stop hotkey!
Now when I hit my stop hotkey nothing happens...

Tags: patch
Revision history for this message
Nicolò Chieffo (yelo3) wrote :

protected override void PluginInitialize()
{
    special_keys = new SpecialKeys();
    special_keys.Delay = new TimeSpan(500 * TimeSpan.TicksPerMillisecond);

    special_keys.RegisterHandler(OnSpecialKeysPressed,
    SpecialKey.AudioPlay,
    SpecialKey.AudioPrev,
    SpecialKey.AudioNext
    );
}

as we can see the stop hotkey is even not considered...
please add it!

Changed in banshee:
status: Unconfirmed → Confirmed
Revision history for this message
Cesare Tirabassi (norsetto) wrote :

Just for my personal culture.
Would adding SpecialKey.AudioStop to the list in MMKeysPlugin.cs make it work?
If so, why not proposing a patch upstream? Perhaps it is faster :)
Beside your joke, is there any reason you know why this MM key is disabled by default in gnomeconf?

Thanks.

Revision history for this message
Nicolò Chieffo (yelo3) wrote : Re: [Bug 99817] Re: [feisty] stop hotkey not working

I just know that every gnome app starts without this key considered.
rhythmbox upstream told me that they don't want to have the stop
feature, so they decided to map pause to the stop hotkey

Changed in banshee:
status: Unknown → Confirmed
Revision history for this message
Chris Halse Rogers (raof) wrote :

Here's a (currently untested) debdiff to fix this. Please test it out, see if it fixes it for you.

I'll test it out this evening.

Revision history for this message
Nicolò Chieffo (yelo3) wrote :

can you upload it upstream?

Changed in banshee:
status: Confirmed → Fix Released
Emmet Hikory (persia)
Changed in banshee:
assignee: nobody → raof
Revision history for this message
Wouter Stomp (wouterstomp-deactivatedaccount) wrote :

This was fixed upstream and should be in ubuntu now.

Changed in banshee:
status: Confirmed → Fix Released
Changed in banshee:
importance: Unknown → Low
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.