Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: [PATCH] Minor wps.c changes
From: Justin Heiner (jheiner_at_yahoo.com)
Date: 2002-09-07


--- Magnus Holmgren <lear_at_algonet.se> wrote:
> Hi,
>
> The attached patch fixes a couple of small details in wps.c and adds
> some information in the player ID3 info screen. The following things are
> changed:
>
> * Make sure wps_refresh and status_draw are called frequently (i.e., a
> couple of times per second) when changing volume and showing the ID3
> info screen, so that things that should update (play time, charging
> indicator) really do update.

Seems like a goodidea

> * When an USB cable is detected, turn off various icons on the display.

Again, agreed

> * In the player ID3 info screen, the "header screen" ("-ID3 info-") is
> removed, since it's not important and I find it annoying. :) Also, added
> track number and playlist position/size. Also, corrected the frequency
> text (it shows Hz, not kHz).

I think having that displayed at least for a tiny bit isn't that bad. I think
that it's good sometimes to have an indicator of where you are, if it doesn't
severely detract from the use of the device. It's good for people who don't
know the device very well.

> * Turn off the (player) audio icon when going to the menu.

Another one that I think should stay. The player icon being there means that
you're in a menu while playing something. I think it gives more information
about the status of the device, which is never a bad thing.

-Justin

__________________________________________________
Do You Yahoo!?
Yahoo! Finance - Get real-time stock quotes
http://finance.yahoo.com



Page was last modified "Jan 10 2012" The Rockbox Crew
aaa