After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 311695 - Wishlist : Prev/Next track hotkey
Wishlist : Prev/Next track hotkey
Status: RESOLVED FIXED
Product: sound-juicer
Classification: Applications
Component: interface
2.11.x
Other Linux
: Normal enhancement
: ---
Assigned To: Sound Juicer Maintainers
Sound Juicer Maintainers
: 311592 311696 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2005-07-27 09:37 UTC by Lionel Dricot
Modified: 2005-08-01 17:32 UTC
See Also:
GNOME target: ---
GNOME version: 2.11/2.12

Attachments
Hack patch to implement skipping (10.12 KB, patch)
2005-07-29 08:53 UTC, Ross Burton 		none Details | Review

Description Lionel Dricot 2005-07-27 09:37:31 UTC 
I would be great to have the ability to play prev/next track of the CD.

A button will maybe not fit in the interface, so I suggest just two entries in
the  "Disc Menu" (or what about create a "play" and a "ripping" menu ?), with
hotkeys.
Comment 1 Ross Burton 2005-07-27 09:44:29 UTC 
*** Bug 311696 has been marked as a duplicate of this bug. ***
Comment 2 Ross Burton 2005-07-27 09:45:08 UTC 
Fancy making a patch?
Comment 3 Ross Burton 2005-07-29 08:53:12 UTC 
Created attachment 49919 [details] [review]
Hack patch to implement skipping

Attaching a bit of a hacky patch for skipping, but it works.  Really sj-play.c
needs a refactor but a bit of copy-and-paste never hurt anyone.

C-n and C-v are next and previous shortcuts resp.
Comment 4 Bastien Nocera 2005-07-29 08:57:25 UTC 
Isn't Ctrl+V already used for pasting in the text entries?
Ctrl+B seems like a better option.

Also, you might want to implement this little thing Iain seems to be keen on:
if (direction == PREVIOUS && time < 5 seconds)
    seek_back_to_beginning_of_the_track();
Comment 5 Ross Burton 2005-07-29 09:07:16 UTC 
B for Back. I'll blame not noticing that on a brainfart.

Shouldn't that test be time > 5 seconds?
Comment 6 Bastien Nocera 2005-07-29 09:13:42 UTC 
Yeah, not enough caffeine yet.
Comment 7 Ross Burton 2005-08-01 10:13:44 UTC 
Added to CVS.
Comment 8 Ross Burton 2005-08-01 17:32:15 UTC 
*** Bug 311592 has been marked as a duplicate of this bug. ***