FS#6506 - Correct battery level display

Attached to Project: Rockbox
Opened by Tobias Heimann (Killaruna) - Friday, 05 January 2007, 00:16 GMT
Task Type Patches
Category Themes
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


In the current build, the highest level battery icon in the WPS is only displayed for a level of 100. This patch corrects the problem and assigns the highest level an interval as all others.
This task depends upon

Closed by  Nicolas Pennequin (nicolas_p)
Friday, 13 April 2007, 15:17 GMT
Reason for closing:  Out of Date
Additional comments about closing:  The new WPS code does this.
Comment by Eddy (bascule) - Friday, 05 January 2007, 09:11 GMT
I made an assumption that it was the designed behaviour, as with the volume enum.

It makes sense to me for maximum volume (0db) to have it's own state, because it represents the line-level output on those targets such as the H1x0 that has a variable level output on the line-out jack.

i.e., I have my volume indicator set in my WPS as:
%?pv<mute|v.v. low|v. low|low|med|high|v. high|v.v. high|line>

For the battery, you could argue that you may want a 'Full' or 'Max' indicator for the 100% value.
Comment by Paul Louden (darkkone) - Sunday, 07 January 2007, 00:23 GMT
Especially in the case of charging, being able to see 100% on its own is handy.

For WPSes one could readily use the same image for the top two cases so that it appears full, thus mimicing something like this behaviour without a patch, and still allowing the original behaviour in the original case.