PDA

View Full Version : Previous / Next buttons for "Browse Atrwork"



egd
2006-05-06, 06:45
When browsing artwork in 6.5x (and earlier versions) slimserver provides a numeric and alphabetic list "
1 2 A B C D E F G H I J K L M N O P Q R S T U V W Y Z
" at the top and bottom of the screen for navigating.

What would be great is perhaps <...> as well when < & > select previous and next albums in sequence - saves the user having to see which letter is currently bold.

egd
2006-05-15, 03:17
When browsing artwork in 6.5x (and earlier versions) slimserver provides a numeric and alphabetic list "
1 2 A B C D E F G H I J K L M N O P Q R S T U V W Y Z
" at the top and bottom of the screen for navigating.

What would be great is perhaps <...> as well when < & > select previous and next albums in sequence - saves the user having to see which letter is currently bold.

Take it the general consensus is this is a crappy idea?

JJZolx
2006-05-15, 13:19
Take it the general consensus is this is a crappy idea?
No, it's been discussed before. I think it would be great as well. Probably low priority right now, and it may not be that easy to do.

http://forums.slimdevices.com/showthread.php?t=20199#post79956

Grotus
2006-05-15, 21:45
JJZolx blurted out:
> egd Wrote:
>
>>Take it the general consensus is this is a crappy idea?
>
> No, it's been discussed before. I think it would be great as well.
> Probably low priority right now, and it may not be that easy to do.
>
> http://forums.slimdevices.com/showthread.php?t=20199#post79956

This should be totally doable in a skin. All you really would need to
do is set pageprev and/or pagenext to the item number where the previous
and next page begin, before the pagebar block gets processed.

Calculating those values is a little trickier, but the information to do
so is available to the skin in pageinfo.alphamap.

I'd probably do something like:

[% IF pageinfo.alphamap;
alphapages = pageinfo.alphamap.values.nsort.unique;
FOREACH alphapage = alphapages;
IF alphapage == pageinfo.startitem;
pageprev = loop.prev();
pagenext = loop.next();
END;
END;
END; %]

If that works, then we can make that into a block and put it in
hreftemplate to make it easier for skin authors to use.

Grotus
2006-05-16, 22:17
Robert Moser blurted out:
> I'd probably do something like:
>
> [% IF pageinfo.alphamap;
> alphapages = pageinfo.alphamap.values.nsort.unique;
> FOREACH alphapage = alphapages;
> IF alphapage == pageinfo.startitem;
> pageprev = loop.prev();
> pagenext = loop.next();
> END;
> END;
> END; %]
>
> If that works, then we can make that into a block and put it in
> hreftemplate to make it easier for skin authors to use.

Ok, it's in hreftemplate now as alphapagecalc. I didn't actually add
the buttons to any of the skins, but if skin authors/users want to do
it, they can change the following line in the pagebar block in
cmdwrappers from:
[%- IF NOT pageinfo.alphamap; PROCESS pagebarcalc pagesperbar = 15; END %]
to
[%- IF pageinfo.alphamap;
PROCESS alphapagecalc;
ELSE;
PROCESS pagebarcalc pagesperbar = 15;
END %]

Or the PROCESS alphapagecalc can be placed before the PROCESS pagebar in
one of the pages.