This is the bug/patch tracker for Rockbox. Click here for more information.
Quick links: Bugs · Patches · Rockbox frontpage
FS#12213 - Cabbie v2 for 128x64x1 (sansa clips): show volume in dB when changing volume
Attached to Project:
Rockbox
Opened by Bertrik Sikken (bertrik) - Sunday, 31 July 2011, 10:29 GMT+2
Last edited by Bertrik Sikken (bertrik) - Tuesday, 21 February 2012, 10:06 GMT+2
Opened by Bertrik Sikken (bertrik) - Sunday, 31 July 2011, 10:29 GMT+2
Last edited by Bertrik Sikken (bertrik) - Tuesday, 21 February 2012, 10:06 GMT+2
|
DetailsThis patch shows the volume numerically in dB for about 1 second when changing volume.
This behaviour was in the clip wps before it was merged with the wps from the iriver remote. I liked it very much, so this patch brings it back. |
This task depends upon
Closed by Bertrik Sikken (bertrik)
Tuesday, 21 February 2012, 10:06 GMT+2
Reason for closing: Out of Date
Additional comments about closing: Needs to be coordinated with all cabbiev2 wps'es, will probably never happen.
Tuesday, 21 February 2012, 10:06 GMT+2
Reason for closing: Out of Date
Additional comments about closing: Needs to be coordinated with all cabbiev2 wps'es, will probably never happen.
Initially the clip wps was copied from the irivier remote 128x64x1 wps and modified specifically for the clip to avoid any viewport crossing the yellow/cyan boundary (which looks very ugly). Later, during devcon 2010, this clip-specific wps was merged back into the generic 128x64x1 WPS. During this merge, it also lost the "show numerical volume" feature, that was apparently present in the wps that the clip-specific wps was originally based on.
I didn't realise until now that this feature is not present in any other cabbiev2 wps for different targets.
I still think this is a very useful feature, I often find myself switching back to the list view, just to see the exact volume level.
I've basically copied the %mv line instead of the %pv line.
I can't use patch so maybe someone could create a patch based on these files ?
I've tested most of them and seems to work as advertised.
The default white colour of the numeric volume looks quite "off" compared with the rest of the (mostly yellow) cabbiev2 theme. So use yellow for this too?
Also, I think we should center the numeric volume in the space currently occupied by the volume bitmap. Horizontal is easy, just use %ac (align centre). To align vertically, we might need to define a new centered viewport with the numeric volume, and switch with %?mv<Vd(x)|Vd(y)> between numeric and bitmap.
Kugel, I didn't provide a patch because I don't know how.
The enthusiasm is great...but this is really one of those "do it properly" things...
This is, after all, "the face of Rockbox".
For many players, simply wrapping "%?mv<%pvdB|.....>" around the existing volume declaration is not sufficient although it "just works".
What this really needs is:
- two conditional viewports, one for the graphical volume, on for the textual volume display. (so that the textual display can be properly aligned)
- a simple conditional statement to flip those viewports ("%?mv<%Vd(textual)|%Vd(graphical)>" for example)
and for many players of larger(ish) screen resolution...
- A larger font loaded for the textual viewport to give the volume display more "visual impact" and more closely relate to the size of the icon it is temporarily replacing.
I mentioned in IRC when this topic came up that I am happy to do the work, I just cannot do it *right now*...as soon as I get my RaaA theme into SVN I shall look into doing this, as it seems to be wanted in general and I'm more than comfortable with Rockbox skin syntax. There are many more optimizations that I wish to include in SVN cabbiev2* and while doing those, this would be simple to add.
A note about the text colour, for colour targets the foreground colour should be a mid-to-light grey (CCCCCC to be exact)...definitely not white. If colour targets do have white text in the .wps, .fms, .rwps, .rfms, lists or menus then this is something I would consider an error.
[St.]
* -Translated .wps, .fms, .rwps and .rfms "titlebars"
-Increased visual similarity between .wps, .fms, .rwps and .rfms
-"progressbar" style volume and battery indicator icons as opposed to bitmap strips
-Some tweaks to the displayed metadata (TBD) to provide as much info as possible in cases of missing metadata with conditional placement to eliminate gaps caused by empty fields (taken from my current RaaA cabbiev2 project))