Using Viewports with Alternating lines

[robin0800]

%?C<%t(10)%Vd(a)%Vd(c)|%t(5)%Vd(b)%Vd(c)>;%t(10)%Vd(d)

This code snippet is from from the MaximumArt Theme for the C200.
It  does not now work, it appears the alternate line mechanism can't now parse viewports.
Is this  correct?

[JdGordon]

doesnt work how? it should work and was only a week or two ago.

[robin0800]

It appears to parse the first term correctly but the second term is then put over the top and the alternating function then just stops so nothing gets updated.

[JdGordon]

Upload the wheole zip

[robin0800]

Rename txt to zip to use

This is also available on the themes Site

_{SOAP EDIT:  Removed attachment.}

[JdGordon]

file is corrupt.. link to the theme site

[robin0800]

http://themes.rockbox.org/index.php?themeid=240&target=sansac200

this should be ok

[[Saint]]

%?C<%t(10)%Vd(a)%Vd(c)|%t(5)%Vd(b)%Vd(c)>;%t(10)%Vd(d)

It's not going to magically fix it, but I just noticed that displaying viewport c for both the true and false conditions is less than pointless.

If it's going to be displayed all the time, it needn't be in any form of condition at all.



[St.]

[soap]

Rename txt to zip to use

This is also available on the themes Site

The reason .zip files (and others) are not accepted by the forum software is because they clog up the forum software.
Please do not attempt to bypass this by renaming files.

[audio-i]

%?C<%t(10)%Vd(a)%Vd(c)|%t(5)%Vd(b)%Vd(c)>;%t(10)%Vd(d)

It's not going to magically fix it, but I just noticed that displaying viewport c for both the true and false conditions is less than pointless.

If it's going to be displayed all the time, it needn't be in any form of condition at all.



[St.]

The last alternating subline doesn't display viewport c.

What's complicated about the way it's done (in order to write the wps code differently) is that the AA present and no AA present sequence times are different. And even with a work around for this problem, the theme seems to show other problems under the current parser, that do not break the theme but that makes it behave weird

[[Saint]]

There's different ways to do this though, I'm not sure why you can't see that viewport c is true all of the time and will be constantly displayed but just drawn over by whatever is in viewport d.

This can, and should be avoided to prevent update issues.

You should really have separate %?C<|> statements inside viewports a and b

I'll need to look at the code to make sure that's correct though.



[St.]

[audio-i]

I'm not sure why you can't see that viewport c is true all of the time and will be constantly displayed but just drawn over by whatever is in viewport d.

%?C<%t(10)%Vd(a)%Vd(c)|%t(5)%Vd(b)%Vd(c)>;%t(10)%Vd(d)

Viewport d is in an alternating subline sequence, preceded by ";" and "%t(10)". I think the theme author's intention is:

AA present: display 10 secs viewports a and c, display 10 secs viewport d

AA not present: display 5 secs viewports d and c, display 10 secs viewport d

[[Saint]]

Viewport d is in an alternating subline sequence, preceded by ";" and "%t(10)". I think the theme author's intention is:

AA present: display 10 secs viewports a and c, display 10 secs viewport d

AA not present: display 5 secs viewports d and c, display 10 secs viewport d

I can see that, but what *really* happens is that viewport c is drawn 100% of the time, and viewport d I assume covers it up.

If none of those viewports a, b or d are supposed to cover up c either in part or in full, then there's no point in c being in a conditional statement at all...which is why I assume the intent is to cover the viewport from view instead of simply not displaying it in the time that it's not needed.

Probably written when viewports weren't cleared correctly.

I guess I'll need to look at the code if I want to show a way this can be done with conditional statements that work.


EDIT:

Argh! Right...the penny dropped with me, yes...it is on a conditional subline but I'm not sure it it will be being cleared properly or not?

I'll try map out the way I'd do it if I get a chance to later on.

I'm hoping it'll be easier to understand, and even work if I'm lucky, once I get to looking at the code finally.

[St.]

[robin0800]

audio-i is correct this is what I wanted and used to work.

Note to St
viewport "a" draws album art viewport "b" draws no album art in the same viewport
Viewport "c" draws all the rest of the viewports for that screen.
Viewport "d" should draw different viewports on a new screen


Edit

Can someone confirm whether the alternate line problem is a bug? if so I will post it as such.