Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: the UI viewport logic.. someone please help my understanding?

Re: the UI viewport logic.. someone please help my understanding?

From: Thomas Martitz <thomas.martitz_at_student.htw-berlin.de>
Date: Wed, 28 Oct 2009 22:14:18 +0100

Jonathan Gordon schrieb:
> 2009/10/28 Thomas Martitz <thomas.martitz_at_student.htw-berlin.de>:
>
>> Why? _set_defaults doesn't, and won't, need to know what screen it's called
>> from. It returns the UI vp (which is in fact default viewport for the whole
>> UI excluding skins), or returns _set_fullscreen if no UI viewport is
>> specified. The caller needs to know that (and it obviously does), and then
>> decides for _set_defaults() or _set_fullscreen().
>>
>>
>
> You just said excluding!!!! Therefore something has to know what it wants back..
>

The caller, yes.

> If we agree that _fullscreen() should return the whole display (which
> means sbs is disabled) , and set_defaults() should return the ui area,
> then when the skin_parser starts parsing (i.e before it sees any %we
> or %wd tags) the default viewport should be setup with a call to
> set_defaults().
>

fullscreen (maybe renamed) should do what it does now, IMO. I fail to
see the point of having a function that respects the classic sb but not
the custom sb. Where would the be useful? You surely want to either be
ultimately fullscreen (0,0,LCD_WIDTH,LCD_HEIGHT), or let background
drawings happen.

> It should only fix that to calling fullscreen() when a %wd tag is found.
>
> Now, how does set_defaults() know that its being called by a skin or
> by the lists? it doesnt, and shouldnt... it should be returning the
> same viewport for both, i.e the one from the setting. therefor makeing
> %Vi redundant and unused.
>

That's what set_defaults() does. Yet I'm skeptical about calling it from
the WPS. The WPS should really be as fullscreen as possible (i.e. so
that it just doesn't draw over the sbs/classic statusbar) since it
doesn't draw any list and the UI vp doesn't make any sense due to the
customizability. That's what _set_fullscreen() achieves.

> I *will* accept that %Vi can be used as a hint to themers as a way to
> say where it is safe to put the ui or other stuff in other skins, but
> in that case a comment would do it better.
>

It's not simply a hint.
Received on 2009-10-28


Page was last modified "Jan 10 2012" The Rockbox Crew
aaa