Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Bugs
  • Category User Interface → Themes
  • Assigned To No-one
  • Operating System Olympus M:Robe 500
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Leo Witt - 2010-12-24
Last edited by Teruaki Kawashima - 2011-01-15

FS#11835 - %Lt tag does not work

Version: r28880

Hello,

the %Lt tag does not work on the m:robe500.

Tested with simulator too.

Closed by  Teruaki Kawashima
2011-01-15 11:45
Reason for closing:  Fixed
Additional comments about closing:  

Fixed in r29058.

Hayden Pearce commented on 2010-12-25 04:14

Can you please post an example of how you are using it?
I have several themes I have developed using this tag in the current build and all is well.
Attach the theme, and I'll have a look at it (or someone else will) when I get a chance.
[St.]

Leo Witt commented on 2010-12-25 10:22

Hi,

my exapmle is very simple and I use this tag with a conditional, like it is described in the wiki.

Hayden Pearce commented on 2010-12-25 13:27

Well, I'm not sure what you're seeing…but I'm also unsure what you're expecting to see.
The code you have here is the same as: "If there's a List Title, display it" which coupled with the positioning you've chosen adds up to "Do absolutely nothing different at all when compared to default behaviour".
What is the effect you're trying to achieve, and what is it you're getting instead?
[St.]

Leo Witt commented on 2010-12-25 15:15

I'm tying to get the effect like the terminal theme has:
http://themes.rockbox.org/index.php?themeid=1067&target=cowond2

But I get instead: nothing.

Am I wrong?

Leo Witt commented on 2010-12-25 15:31

for example:

with

%?Lt<cake|cheese>

I get "cheese" (the hole time) on the m:robe

Jonathan Gordon commented on 2010-12-25 23:39

of course, cheese would be displayed for every screen where there is a list title… you neeed to actrually use %Lt to get it to display the title…

e.g %?Lt<no title|%Lt>

Leo Witt commented on 2010-12-26 00:14

Okay guys,

a picture is worth a thousand words:
(The following code is used on both simulator builds)

%wd

%V(0,0,320,30,1)%Vf(000000)%Vb(00eaff)
%?Lt<%Lt|cheese>

%Vi(-,10,40,300,100,1)%Vf(00eaff)%Vb(000000)

I hope, you understand my concern now.

Leo Witt commented on 2010-12-26 00:24

I'm also asking me, why the [red arrow] does not disappear like on d2.

Thanks!

Hayden Pearce commented on 2010-12-26 05:13

I'm assuming you don't want the title-bar…which is what I was saying earlier, but perhaps not so clearly without spelling it out.

All you need to do (and I know it looks fucking weird but I assure you it works) is:

%?Lt<>

Loosely translated: "If there's a list title (which is always true afaik), do nothing".

If you *do* want the title bar to display, then:

%?Lt<%Lt>

Will work fine.

Teruaki Kawashima commented on 2010-12-26 07:02

this seems to fix the bug.

Leo Witt commented on 2010-12-26 12:11

Hayden,

all I want is to *move* the title bar to an other position, like on the terminal theme for the d2.

Teru,

it's a pitty, but I can't build rockbox on my own…

Hayden Pearce commented on 2010-12-27 07:28

Leo,

Ok, that's fine…easy.

To do this, we'll need at least two viewports…but we know this much already.
Firstly, we'll want the viewport that houses our list title-bar (lets assume the screen is 100x100px):

%V(0,0,100,20,1)%Vf(000000)%Vb(FFFFFF)
%ac%?Lt<%Lt>

Then, we want the rest of it to be the UI viewport, and we don't want either to overlap so:

%Vi(-,0,20,100,80,1)%Vf(FFFFFF)%Vb(000000)

It gets more interesting if you want to display the list title-bar icon, but you haven't yet mentioned wanting to do so,
so I'll leave that for now.

This works as expected on my end.

Teru,

What does your patch actually fix? As far as I'm concerned, it is working fine.

Just some confusion as to how the tag should be used I think.

When %?Lt condition is used, the list title-bar is not drawn in the UI viewport.
-To display the title-bar elsewhere, another viewport is defined and "%?Lt<%Lt>" is used.
-To remove the title-bar from display in the UI viewport, you can do "%?Lt<>" (if there's a title, do nothing).

I never find the need for a "false" case in the %?Lt condition, as I don't think it would ever be true.
I'm pretty sure that all screens have a list title, no?

Leo Witt commented on 2010-12-27 11:57

Hayden,

thanks for your help, but do you already tested this with mr500-sim ?

Check it, and you will see ;-)

This is obviously a bug and not a wrong usage of the tag.

Teruaki Kawashima commented on 2010-12-27 12:38

The patch fixes that %Lt tag doesn't work on targets which have more than 1 screen.
almost all screens using list seems to have title but there are screens not using list. those do not have list titles and the "false" case might be used in such case. e.g. quick screen.

Hayden Pearce commented on 2010-12-29 13:04

Leo,

Can you actually post the code you're using please?

[St.]

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing