Support and General Use > Theming and Appearance Customization

Themes using Album Art don't work in RB on iRiver H10 20GB

<< < (2/3) > >>

Chronon:
I don't think that was snarky.  Regarding "drive-by skinners" it's just not wise to count on the creator of a given theme to make the desired changes on any particular time frame.  There is a whole list of reasons that may delay or prevent the creator of a given theme from making the changes that you want.

You say that "somebody" should fix them.  That somebody could very well be you.  This is a community effort and anyone who wants to help is wholeheartedly welcome to do so.  Pointing at something that you feel needs to be done and just saying "someone else should take care of that" could be seen by some as expressing some feeling of entitlement.  

I'm sure that the themes will be updated.  But it will take time.  We can all speed this up by updating the wiki with updated WPS code.  I have seen some users already in the process of updating themes for their respective players.

The recent changes in WPS related code include the inclusion of Album Art and new scrolling margins tags.  I have found that it's mostly the scrolling-margins related tags that have caused problems in my themes due to a change in syntax from the original scrolling-margins patch.  Themes that only required the Album Art tags seem to mostly work (except for those with dynamic tags on the same line as the album art).  I have edited some of the (Gigabeat) themes with (scrolling) margins to work with the new syntax and I will update these when I have the time.

FuzzplugJones:
Definitely was snarky, as I said I assumed WPS developers were here and active; your gallery clearly states that I cannot upload someone else's stuff and so clearly I/you/anyone cannot fix these things for the benefit of the community.  Please don't be like so many open source projects where the answer to everything is "fix it yourself" when someone might be coming in here wondering if it's already somebody's job and the desire is not to reinvent the wheel.  I'm inside a WPS right now in a text editor and adding %Cl and %C with the proper options doesn't fix it either, rockboxui --debugwps doesn't output anything, and I'm basically lost.  I also assumed mistakenly that there were already working WPSs with AA support since it had been coded into RB and theoretically somebody had to test it.  I incorrectly assumed that I could take advantage of the new code without becoming a developer myself.

So I'll see if I can fix this but as I said, not even the development tools are working properly.  I don't know C so sadly someone else WILL have to fix that.

Chronon:
I could also call your "drive-by skinners" comment snarky, but that doesn't get us anywhere useful, does it?

Many of the themes are published with open licenses and can be modified by anyone.  If the license is not known then modification would be a violation of the creator's copyright.  Don't take someone else's theme and publish it.  That's a copyright violation.  But a theme published under an open license may certainly be updated or modified under the terms of the license.  Someone correct me if I am misinterpreting somehow.

Core developers sign themselves up for certain things.  But themes are not even really a part of Rockbox.  They are user contributed add-ons.  As such there's no formal concurrent versioning or anything of that sort for them.  If you make some changes then contribute them back, if allowed.  And really, modifying a couple of tags isn't that much work.  I have never actually used --debugwps in rockboxui, so I don't have any comment on that.  The developers might be interested in that.

Now, as I said in my post, it's most likely not the album art tags at all that are causing you problems as the syntax for those tags has not changed between the old patch and what is now in SVN.  The syntax for the scrolling margins tags has changed a bit, so themes that used the old scrolling-margins patch will not work currently without updating the syntax to comply with the currently supported tags (search your WPS code for %s and %m).  That has been my experience with getting many themes to display properly on my Gigabeat.

FuzzplugJones:
In this particular one, it seems to be the %e tag that's causing the skin to fail.  Trying to research that, however this forum doesn't let me search for "%e," it instead returns every forum post with at least one "e" in it.

Chronon:
That tag is not supported in SVN.  You still need a patched build for that.

Navigation

[0] Message Index

[#] Next page

[*] Previous page