Support and General Use > Theming and Appearance Customization
Help me debug my theme port!
audio-i:
--- Quote from: [St.] on January 16, 2011, 10:54:03 PM ---There has not been a need to place %vb/f() tags in the viewport definition for quite some time now, this is by design, so that one can easily change the colour of elements and not have it irrelevantly tied to the viewport definition.
--- End quote ---
I don't know why you mention this, this has not been questioned
--- Quote from: [St.] on January 16, 2011, 10:54:03 PM ---
--- Quote from: audio-i on January 16, 2011, 07:59:36 PM ---I got the initial version of the ported themes working on a friend's nano, and I remember that the specific problem gbl08ma is referring to, is because most or all of the viewports foreground colors (%Vf) were defined in a separate line, so correcting this should solve it.
--- End quote ---
No.
...
It is perfectly fine to have them where they are in the code.
--- End quote ---
Are you aware of problems like this one? http://forums.rockbox.org/index.php/topic,26858.0.html
I didn't say there were no overlapping viewports, I just said that the problem the OP describes, can be solved as I explained. If you are going to write an absolute "No" there, you could at least try things first (for the clarity of the OP at least)
Try changing:
--- Code: ---# Track
%V(76,35,92,15,3)
%Vf(ffffff)
%t(6.0)%s%al%?it<%it|%fn>;%t(12.0)%al%?it<%it|%fn>
--- End code ---
to
--- Code: ---# Track
%V(76,35,92,15,3)%Vf(ffffff)
%t(6.0)%s%al%?it<%it|%fn>;%t(12.0)%al%?it<%it|%fn>
--- End code ---
Edit: gbl08ma, I just noticed that when battery left is about < 14% there's no battery icon; you should either put the empty battery in the background, or add it to the battery image strip
gbl08ma:
Thanks for all the help guys. I'm at school using my mobile to connect to the internet, and so I'll read your posts better and aply your suggestions when I get home.
In fact, on the theme editor some viewports look too big, but I left them that way with the fear that text would be cropped if I reduced the size of viewports. I will change that to see if it makes any difference.
[Saint]:
--- Quote from: audio-i on January 16, 2011, 11:50:57 PM ---
--- Quote from: [St.] on January 16, 2011, 10:54:03 PM ---There has not been a need to place %vb/f() tags in the viewport definition for quite some time now, this is by design, so that one can easily change the colour of elements and not have it irrelevantly tied to the viewport definition.
--- End quote ---
I don't know why you mention this, this has not been questioned
--- End quote ---
You yourself specificly stated that you believed this to be the cause of the problems in the theme...I can quote you again if you'd like...but I'm sure you can read a few posts down.
By the way...what makes you think I didn't test it?
There's no point in fixing anything else if you're going to have overlapping textual viewports with scrolling text, if the text is absolutely static then sometimes you can get away with it...but it's bad practise to do so in general.
Testing on target with sane viewports (and no other changes) results in behaviour as expected here.
EDIT: I can only debug problems that I can see, at this point the theme in question (with corrected viewports) works as expected for me.
I can't say conclusively that there is no issue with this part of the theme syntax, just as I can't say conclusively that there is no issue with any other part of the syntax...but I can say that I am not experiencing these troubles in the sim, or on device.
It is entirely possible that there is a problem here, but unless I can actually see it for myself I can't say that it exists.
[St.]
evilnick:
You seem to be both arguing the same point, that the original theme that this was based on used a deprecated way of defining the colour.
[Saint]:
--- Quote from: evilnick on January 17, 2011, 11:38:38 AM ---You seem to be both arguing the same point, that the original theme that this was based on used a deprecated way of defining the colour.
--- End quote ---
Neither way is deprecated, both are perfectly acceptable syntax.
[St.]
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version