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: FS#12321 - Touchscreen: List line padding, to more easily select lines

Re: FS#12321 - Touchscreen: List line padding, to more easily select lines

From: Jonathan Gordon <jdgordy_at_gmail.com>
Date: Sun, 9 Oct 2011 10:12:24 +1100

On 9 October 2011 07:43, Torne Wuff <torne+rockboxdev_at_wolfpuppy.org.uk> wrote:
> On 8 October 2011 11:47, Jonathan Gordon <jdgordy_at_gmail.com> wrote:
>> On 8 October 2011 05:18, Thomas Martitz <kugel_at_rockbox.org> wrote:
>>>> * Automatic (default, line height calculated using a lcd dpi aware
>>>> function)
>>
>> 1) this can already be done in the themes
>
> The automatic setting *can't* currently be done in themes, since
> themes are only resolution dependent, not dpi dependent. Android
> devices have a pretty wide range there...
>
> --
> Torne Wuff
> torne_at_wolfpuppy.org.uk
>

Is that a challenge? :) everything about this patch can already be
done in svn, and the configuration can be added. This is *not* a
useability thing because it is only going to work reliably (err, at
all) for themes which *dont* use skinned lists (which hopefully wont
be very many once themers get the hang of them).

On 9 October 2011 07:31, Frank Gevaerts <frank_at_gevaerts.be> wrote:
> 1) While it's true that there has been work in that direction, I don't
>   think it's entirely fair to say it can be done today. Skinnable
>   viewports still have some issues, and I'm not convinced we have all
>   infrastructure in place to fully implement the default lists.
>

nothing impossible to fix.

Anyway, dead horse beaten. This patch adds nothing which cant already be done.
Received on 2011-10-09


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