FS#8856 - WPS viewport definition: allow background color to be non-transparent even with enabled backdrop

Attached to Project: Rockbox
Opened by Matthias Mohr (aka Massa) (mmohr) - Sunday, 06 April 2008, 23:38 GMT
Last edited by Matthias Mohr (aka Massa) (mmohr) - Wednesday, 28 May 2008, 06:29 GMT
Task Type Patches
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


Behaviour of current SVN version:
If you use a background colour in your viewport tag %V, it'll only be used when no backdrop has been enabled.
When a backdrop is enabled, the defined background colour is ignored and the colour of the backdrop at that position will be used instead.

This patch extends the viewport WPS-Tag with an optional parameter to use the defined background colour even when a backdrop is used or to keep the behaviour as is.

The complete syntax of the extended viewport definition is:

You're allowed to use any of the following variations:

In all above variations bg_use_backdrop will be set to 1;
If a backdrop is present, the defined background colour will be ignored and the backdrop colour will be used instead.
This is how the current SVN version works...

bg_use_backdrop will be set to 0;
Even when a backdrop is enabled, bg_pattern will be used as background colour.

Note: the whole feature is (currently) only implemented for 16-bit colour targets!
Other targets won't be affected.

Note2: using bg_use_backdrop=1 will always display the backdrop as background, even when viewports overlap (which is btw. not officially allowed atm.).
BTW, this behaviour is exactly as it is now - I just want to mention it ;)

The patch is made against SVN version 16999...
Comments are welcome!

This task depends upon

Closed by  Matthias Mohr (aka Massa) (mmohr)
Wednesday, 28 May 2008, 06:29 GMT
Reason for closing:  Invalid
Additional comments about closing:  the whole idea has to be reconsidered...