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: Center on loaded theme FS#10391 (+ set_file bugfix FS#10392)

Re: Center on loaded theme FS#10391 (+ set_file bugfix FS#10392)

From: Jonathan Gordon <jdgordy_at_gmail.com>
Date: Mon, 29 Jun 2009 10:03:36 -0700

2009/6/29 Al Le <al.le_at_gmx.de>

> On 29.06.2009 18:13, Jonathan Gordon wrote:
>
> It shouldn't be very complicated. We already know what settings are
>> theme
>> settings (for the "Save theme settings" menu option).
>>
>> Changing any of those would mark a global flag as dirty. Loading a cfg
>> from the themes menu would reset it.
>>
>> If dirty is set we could do something like showing the last loaded, but
>> with a "*" at the end (for modified), or show a special menu entry like
>> "<modified: based off theme XXX>")
>>
>>
>> That doesnt solve the problem where 2 themes are loaded and both work fine
>> together... i.e on the hxxx's you could load a .cfg for the main lcd and a
>> .cfg for the remote.... neither should be selected.
>>
>
> You exploit the fact that, technically speaking, a theme ist just a .cfg
> file. But it's more. We have some settings that are called "theme settings",
> as Thomas said. Only those settings would be considered when deciding
> whether the theme has been modified. Otherwise there is no sense in having
> the special menu "Save theme settings" (or what's the name?).
>

That is exactly my point... there is nothing special about theme .cfg
files.... Other than them being in the themes directory. They shouldnt be
treated any differently to regular file browsers.
Received on 2009-06-29


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