Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 104 Deprecated: The each() function is deprecated. This message will be suppressed on further calls in /sites/rockbox.org/flyspray/adodb/adodb.inc.php on line 845 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.user.php on line 111 FS#7628 : Non-intuitive keypress behaviour when setting song rating

Rockbox

Tasklist

FS#7628 - Non-intuitive keypress behaviour when setting song rating

Attached to Project: Rockbox
Opened by Eddy (bascule) - Monday, 20 August 2007, 14:21 GMT
Last edited by Eddy (bascule) - Tuesday, 20 November 2007, 21:45 GMT
Task Type Bugs
Category Settings
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

When setting a rating for a song using the WPS context menu, the behaviour is as follows:

LEFT keypress - no action
RIGHT kepress - increment rating by one (which wraps to zero above 10)

Intuitively, LEFT should exit the context menu as per all other context menu entries.

Alternatively, it should decrement the rating by one.

I see no reason why LEFT should not exit the menu, given that it currently does nothing.

If it decremented the value, then it could be confusing for some users to have to navigate up or down in the menu list to then use LEFT to exit the menu.

This is on a Gigabeat, but AFAIK applies to all targets.
This task depends upon

Closed by  Eddy (bascule)
Tuesday, 20 November 2007, 21:45 GMT
Reason for closing:  Fixed
Additional comments about closing:  Now fixed in SVN:
http://svn.rockbox.org/viewvc.cgi?view=r ev&revision=15529
Comment by Steve Bavin (pondlife) - Monday, 20 August 2007, 15:35 GMT
I agree, LEFT should exit.
Comment by Robert Kukla (roolku) - Tuesday, 21 August 2007, 01:17 GMT
It used to, maybe the function got lost in the transition to the new menu api. Maybe it is worth considering using the standard setting api for it (select from a list of values)?
Comment by Steve Bavin (pondlife) - Tuesday, 21 August 2007, 06:29 GMT
Yes, consistency is good.
Comment by Nils Wallménius (nls) - Tuesday, 21 August 2007, 13:21 GMT
+1 for the list suggestion

Loading...