Support and General Use > Theming and Appearance Customization
Allow upto 512 images on a WPS
Llorean:
As was said above, %: should not be needed. If someone can't learn the additional syntax for the one tag, then they can avoid using that tag.
As for designer readability: In the end, should the focus be on "make the WPS easier for designers to make" or "reduce the cost of using pretty WPSes to the end user"?
A complex WPS with many images can add a measurable time to the boot. I don't know the maximum time it can add, but the difference can be noticed by a casual user.
fml2:
--- Quote from: Llorean on June 21, 2007, 09:01:25 AM ---As for designer readability: In the end, should the focus be on "make the WPS easier for designers to make" or "reduce the cost of using pretty WPSes to the end user"?
--- End quote ---
IMHO we should consider both. An extremely bad state on either side should be avoided.
But, as I already said, I don't use a WPS stuffed with images so my opinion is only based on purity and aesthetics.
knight4led:
I was not opposed to using : as a seperator, nor the addition of a %: tag down the road should it be necessary. I was simply stating if the first option is implemented, the second may be either a necessary or a good idea.
Personally I think that if it is decided that this new seperator is good for this tag, maybe it should be used for all enumeration tags that are non-conditional.
Progress in the form of faster loading and more versatility to the WPS parser is, in my opinion, better than trying to force constants based on previous versions of Rockbox.
Yotto:
For the record, I would welcome complexity in WPS code to be able to make nicer WPS's. I've thought of another cool use for this, a smooth circular volume "bar" around... anything. A play button, album art, whatever.
lowlight:
FYI...my patch for enumerated image sets is here: http://www.rockbox.org/tracker/task/7345
If you try it, please, post your comments there.
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version