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 123383 - Add "Rate song" menu item/keybinding
Add "Rate song" menu item/keybinding
Status: RESOLVED OBSOLETE
Product: rhythmbox
Classification: Other
Component: User Interface
unspecified
Other other
: Normal normal
: ---
Assigned To: Sean Harshbarger
Sean Harshbarger
: 167295 311330 338990 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2003-09-27 18:42 UTC by Andy Buckley
Modified: 2018-05-24 10:23 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
add menu items and shortcuts to control volume (4.74 KB, patch)
2006-02-28 10:26 UTC, James "Doc" Livingston
committed Details | Review
Patch to add Ratings Menu item to rate the currently playing song. (3.80 KB, patch)
2007-01-17 21:19 UTC, John Daiker
none Details | Review
Updated for SVN 4749, Added menus for setting rating from 0-5 (6.97 KB, patch)
2007-01-22 07:49 UTC, John Daiker
none Details | Review
Updated for SVN 4749, implemented the way Doc told me to on IRC (5.75 KB, patch)
2007-01-22 11:10 UTC, John Daiker
reviewed Details | Review
Add ratings widget to header... it's to the left of the elapsed time display (8.26 KB, patch)
2007-02-19 21:20 UTC, John Daiker
none Details | Review
Sync the header with the new rating (8.92 KB, patch)
2007-02-19 23:56 UTC, John Daiker
none Details | Review
Show Rating widget in header - 4871 (8.26 KB, patch)
2007-02-28 04:13 UTC, John Daiker
none Details | Review
Updated for latest SVN (5467) (3.53 KB, patch)
2007-12-01 11:00 UTC, John Daiker
none Details | Review

Description Andy Buckley 2003-09-27 18:59:07 UTC
Package: rhythmbox
Severity: enhancement
Version: 0.5.3
Synopsis: Request for improved keybinding support
Bugzilla-Product: rhythmbox
Bugzilla-Component: Interface

Description:
Description of Problem:
Better support for keys-only control of RhythmBox would be nice. In
particular, key-based control of volume and ratings would be useful. As
far as I can tell, this requires a mouse at the moment.

Suggested actions:

Volume control: e.g. Ctrl-<Up/Down Cursor>
Ratings control: e.g. pressing 1-5 when the Rating area is focused




------- Bug moved to this database by unknown@bugzilla.gnome.org 2003-09-27 14:59 -------

The original reporter (buckley@hep.phy.cam.ac.uk) of this bug does not have an account here.
Reassigning to the exporter, unknown@bugzilla.gnome.org.
Reassigning to the default owner of the component, rhythmbox-maint@bugzilla.gnome.org.

Comment 1 Sean Harshbarger 2003-11-02 00:18:50 UTC
I shall look into this issue with the usability guys to find the
proper key combos for this.
Comment 2 James "Doc" Livingston 2005-07-29 09:44:20 UTC
*** Bug 311330 has been marked as a duplicate of this bug. ***
Comment 3 Matthew Sibson 2005-08-13 23:10:42 UTC
In addition, I believe the next and previous track bindings of 'Ctrl-Right' and
'Ctrl-Left' contradict the HIG which states they should be 'Alt-Right' and
'Alt-Left'. This would also fix mice with back and forward buttons that gnome
also seems to assign these keys to.
Comment 4 James "Doc" Livingston 2005-08-15 03:08:13 UTC
*** Bug 167295 has been marked as a duplicate of this bug. ***
Comment 5 James "Doc" Livingston 2005-08-15 03:09:38 UTC
From the gtk 2.8.0 release notes:
"Changes in the tree view widget
 The keyboard shortcuts for header reordering and resizing
 have been changed to (Shift-)Alt-arrows."

This could make using Alt-Right and Alt-Left for next track and previous track
problematic.


Also bug 165716 is related and bug 142378 talk about how most of our key binding
have big issues.
Comment 6 James "Doc" Livingston 2006-02-28 10:26:08 UTC
Created attachment 60306 [details] [review]
add menu items and shortcuts to control volume

This adds menu items and shortcuts to control RB's volume, moving in 10% increments.

Rating tracks by typing numbers will conflict with type-ahead searching (selecting tracks that start with what you type). I can't find the bug number, but having a "Rate Track" submenu is probably the way to go (potentially with shortcuts like Control-3).
Comment 7 James "Doc" Livingston 2006-03-19 08:31:48 UTC
Patch committed to cvs. Retitline bug to reflect remaining request.
Comment 8 John Daiker 2007-01-17 21:19:52 UTC
Created attachment 80547 [details] [review]
Patch to add Ratings Menu item to rate the currently playing song.

Patch created against svn 4742.  Note: Keybindings were created... Ctrl+Right increases the rating, and Ctrl+Left decreases the rating.
Comment 9 John Daiker 2007-01-22 07:49:43 UTC
Created attachment 80862 [details] [review]
Updated for SVN 4749, Added menus for setting rating from 0-5
Comment 10 John Daiker 2007-01-22 11:10:47 UTC
Created attachment 80876 [details] [review]
Updated for SVN 4749, implemented the way Doc told me to on IRC

Sorry about the other patch.  It's amazing what the mind will do with no rest, haha.
Comment 11 Jonathan Matthew 2007-01-28 22:13:05 UTC
I don't think it's a great idea to add a whole new menu for this.  I'm not quite sure where it should go instead - maybe a sub-menu in the Control menu?

I don't really like the idea of using ctrl-left and ctrl-right to modify ratings.  Do we really need app-wide shortcuts for this?  The problem I have here is that unless the rating column is displayed and the playing song is visible in the track list, there's no feedback to the user indicating that anything actually happened.

Being able to modify the rating using keys when a rating widget has focus would be good, though.

Some code style issues:
- rb_shell_player_rating_cb is not a callback, so it shouldn't be named like one
- don't declare variables in the middle of a code block (declaration of 'value' in rb_shell_player_rating_cb)
- spaces before function arguments ("function (args)", not "function(args)")
- it looks like you're inconsistently using spaces and tabs for indentation
- there's not much point clipping score to 0.0 <= score <= 5.0 and then doing 'g_return_if_fail (score >= 0 && score <= 5);'
Comment 12 John Daiker 2007-01-30 00:56:00 UTC
Jonathan,

Thanks for the feedback.  I never really thought about the instances where the rating widget wasn't shown.  In that case it wouldn't make sense to have app-wide shortcuts, haha.

As for this functionality being placed inside of a ratings widget, I like the idea.  With the wasted pixels to the right of the "Title by Artist from Album" label and to the left of the volume control button, would we place a ratings widget up there to handle rating songs.  This would ensure that whether the ratings were shown in the library view, the user would still be able to know that something happened when they used the keybindings.  I'd like to see this happen, but the UI setup for the toolbars, as you know, is vastly different from the UI setup for the menus.
Comment 13 John Daiker 2007-02-19 21:20:32 UTC
Created attachment 82918 [details] [review]
Add ratings widget to header... it's to the left of the elapsed time display

Updated for SVN 4829.
Add ratings widget to header next to elapsed time widget.
Updated code style (tabs, function calls, variable declarations)
Comment 14 John Daiker 2007-02-19 23:56:55 UTC
Created attachment 82924 [details] [review]
Sync the header with the new rating

Updated to sync the Header with the new rating.  This previously did not happen when using the keyboard bindings.
Comment 15 John Daiker 2007-02-28 04:13:30 UTC
Created attachment 83518 [details] [review]
Show Rating widget in header - 4871

Updated for the current SVN 4871
Comment 16 John Daiker 2007-12-01 11:00:54 UTC
Created attachment 99961 [details] [review]
Updated for latest SVN (5467)

Updated to latest SVN (5467)
Removed keybindings (as per #11)
Removed Ratings menu (as per #11)
Changed rb_header_rated_cb to actually update the widget when a song is rated in the browser view.
Comment 17 Jonathan Matthew 2009-04-13 10:58:53 UTC
*** Bug 338990 has been marked as a duplicate of this bug. ***
Comment 18 GNOME Infrastructure Team 2018-05-24 10:23:51 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/rhythmbox/issues/15.