Rockbox

Tasklist

FS#12804 - Custom UI Viewport settings ignored for Pitchscreen and Track Info screen

Attached to Project: Rockbox
Opened by Tomasz Papiez (electric_indigo) - Thursday, 17 January 2013, 15:47 GMT
Last edited by Thomas Martitz (kugel.) - Wednesday, 15 January 2014, 17:59 GMT
Task Type Bugs
Category Themes
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 0
Private No

Details

I am writing a theme for Sansa Clip Zip. In my .sbs I defined two different custom viewports and I want to switch between them depending on current screen. It works in all but two cases - the Track Info screen uses the same viewport as the Context Menu and the Pitchscreen seems to inherit the viewport setting from the WPS. Here is the sample of the code:

%?cs<%VI(small)| # Menus
%VI(small)| # WPS
%VI(small)| # Recording screen
%VI(small)| # FM Radio screen
%VI(small)| # Current Playlist screen
%VI(small)| # Settings menus
%VI(small)| # File browser
%VI(small)| # Database browser
%VI(small)| # Plugin browser
%VI(big)| # Quickscreen
%VI(big)| # Pitchscreen (ignored - it uses the same %VI as WPS)
%VI(small)| # Setting chooser
%VI(small)| # Playlist Catalogue
%VI(small)| # Plugin
%VI(small)| # Context menu
%VI(small)| # System Info screen
%VI(small)| # Time and Date screen
%VI(small)| # Bookmark browser
%VI(small)| # Shortcuts menu
%VI(big)%Vd(status_bar)%Vd(list_title)> # Track Info screen (%VI ignored - it uses the same %VI as Context Menu)

### UI viewports ###
%Vi(big,0,10,96,86,-)
%Vi(small,5,26,91,70,-)

########### end of code ###########

%VI seems to be the only setting which is ignored as the %Vd tags in the given example are respected.
I am currently using the 29e51a1-130112 build for the UI simulator and the fa6dc05-130115 build for the actual device. I didn't examine any other hardware or simulator, but I guess such thing is not hardware-related, so I set "Player Type" in this report to "All players".
This task depends upon

Closed by  Thomas Martitz (kugel.)
Wednesday, 15 January 2014, 17:59 GMT
Reason for closing:  Fixed
Additional comments about closing:  2eb873c
Comment by Hayden Pearce (St.) - Wednesday, 30 January 2013, 09:59 GMT
Try adding a single blank line after each custom UI viewport declaration.

For example

[code]
### UI viewports ###
%Vi(big,0,10,96,86,-)

%Vi(small,5,26,91,70,-)

[/code]
I seem to recall running into this before.

[Saint]

Loading...