Rockbox

Tasklist

FS#10553 - Viewports are not in the manual

Attached to Project: Rockbox
Opened by David Kauffmann (BdN3504) - Friday, 21 August 2009, 23:44 GMT
Last edited by Alex Parker (BigBambi) - Sunday, 20 September 2009, 16:58 GMT
Task Type Bugs
Category Manual
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Version 3.3
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Let's change this. here's a first effort i have been working on for three nights now. Please help me fix the \opts, i can't figure out how to use them in combination with {example}, that's why i put the examples in their own files.
This task depends upon

Closed by  Alex Parker (BigBambi)
Sunday, 20 September 2009, 16:58 GMT
Reason for closing:  Accepted
Additional comments about closing:  Accepted as part of r22756 - thanks!
Comment by David Kauffmann (BdN3504) - Friday, 21 August 2009, 23:44 GMT
oops, forgot the file. here it is.
Comment by David Kauffmann (BdN3504) - Friday, 21 August 2009, 23:58 GMT
fixed this myself. comittable in my eys, you got anything to add?
Comment by David Kauffmann (BdN3504) - Saturday, 22 August 2009, 11:12 GMT
fixed problems as discussed in irc.
Comment by David Kauffmann (BdN3504) - Saturday, 22 August 2009, 11:32 GMT
forgot to remove a paranthesis which broke a table: update!
Comment by Thomas Martitz (kugel.) - Saturday, 22 August 2009, 12:23 GMT
It seems you mixed up charcell and monochrome displays. charcell doesn't have wps-viewports or ui-viewports at all.

I also think that it's a poor choice to have the RTC stuff in the example, the additional file it needs isn't worth it.

I'd like to see an example for an "ordinary" unlabeled viewport as well.
Comment by David Kauffmann (BdN3504) - Monday, 24 August 2009, 08:51 GMT
removed rtc, excluded charcell.
Comment by David Kauffmann (BdN3504) - Thursday, 27 August 2009, 18:32 GMT
Committable?
Comment by Jonas Häggqvist (rasher) - Thursday, 27 August 2009, 21:39 GMT
It's a good idea to search the tracker before writing up something. Anyway,  FS#9934  might have something usable. If not, it should be closed and replaced by this.
Comment by David Kauffmann (BdN3504) - Thursday, 27 August 2009, 22:14 GMT
Good idea indeed. i vaguely remember having searched and did not get any results... well anyways, i have attached a preview, and also updated to the new rbtabular already in last post. please point out any problems, if you find them.
Comment by David Kauffmann (BdN3504) - Thursday, 27 August 2009, 22:22 GMT
forgot a closing paranthesis. broke the table for grayscale targets. update.
Comment by David Kauffmann (BdN3504) - Thursday, 27 August 2009, 22:27 GMT
and the proper preview. man i don't think that there has ever been someone coding so many errors into such a simple document.
Comment by David Kauffmann (BdN3504) - Tuesday, 01 September 2009, 08:41 GMT
Changed the wording slightly in grayscale conditional. Added a line about the end of viewports.
Comment by David Kauffmann (BdN3504) - Wednesday, 09 September 2009, 17:51 GMT
Added the UIviewports part.
Added %V tag to wps tags.
Fixed some minor glitches (no %s after opts, removed the obsolete %P tag in the WPS tags)
Comment by David Kauffmann (BdN3504) - Wednesday, 09 September 2009, 18:00 GMT
Oops, wrong commit last time: somehow copypasta did me wrong.
Comment by Alex Parker (BigBambi) - Wednesday, 16 September 2009, 11:34 GMT
I'll try and have a proper look later, but a couple of immediate thoughts: "'s" is the possessive, "s" is the plural so in my opinion it should be |s not |'s - perhaps we can find a way to rephrase this slightly to avoid the issue :)

fgcolor and bgcolor - I know on the wiki it is like this, but it seems a little odd not to change them to fgcolour and bgcolour - they are just place holders showing you where to put other values, and I can't see any reason to keep them with the US spelling instead of the correct (for the manual) UK..

There are a couple of other minor things like "the blanks fields" should be "the blank fields", but as I say I'll look properly later.

Either way, good effort thus far :)
Comment by David Kauffmann (BdN3504) - Friday, 18 September 2009, 02:20 GMT
thanks for the love.
fixed these small issues. had to think alot about what you were refering to with |'s, simply replaced "-'s" with "hyphens".
Also, if you take a look at it, please take investigate in the repeat states. i took out an opt there because i don't think a-b is restricted to any player. i can be wrong, but i asked this question on IRC several times and no one answered, so i simply took the initiative.
Comment by Thomas Martitz (kugel.) - Friday, 18 September 2009, 07:31 GMT
Taking the initiative is good, but in a separate patch please.

Loading...