Re: Man pages

• Messages sorted by: [ date ][ subject ][ author ]
• Next message: "Patch 10"
• Previous message: Ethan "patches 8 & 9"

Hi,

Here's what I've found.  I think it's probably better to let you do the
actual changes, so that nothing gets lost/repeated:

1.	Animate

line 73: value of "color": perhaps add what "color" is (rgb, I presume,
as defined in the system).

line 94-5: Note that with 0 is a fast line at Width 1 = Note that Width 0
is as fast a line as Width 1?

line 106 ff: "system wide defaults"?  I think the "for all users" is
redundant here.  Maybe not, though.

line 114: sub: suit/suite.  Add "it" to end of line

line 120 sub: its/it's

Also, is there a man page for the autoexec file?  I don't see any mention
in the afterstep man page.  If so, a reference should go in the "see also"
section.

2.	Auto

lines 45-6: the <HR> <SP> the ...


3.	Banner

line 19: is the "Wait" part necessary for start-up?  This should be
explained (I suppose it is in the afterstep man page, though).

(For all the modules, the remark in 1. about the autoexec file goes.)

4.	Cascade

line 36: . . .invoked from (by?) being . . .

line 38: Cascade can be called multiple times?  Didn't know this.  Maybe
point out that this differs from the other modules, which need a different
name to get a different function?

Also, there's no configuration file, right, just command-line options?

5.	Clean

line 18: "from within a menu file by 'Animate'" s/b "... menu file by
'Clean'"

line 78: sub its/it's

lines 95-6: "defines _that_ only iconification ..."  This is obscure to
me.

Also, in the Examples section, you say "At most 3 actions can be
specified".  Do you mean on any line, or in any config file for any one
Clean module?  I'm not sure (and I don't know, either).

line 117: sub suit/suite; add "it" to end of line.

6.	Form

line 29: no copying _or_ pasting functions?

line 165: sub either justified or spread / spreaded

line 222: del "in"

line 254: is that a space after the "and", or just the justification
function of reading a man page?

7.	Ident

line 31: "it pops up a (new? -nother?) window . . ."

line 45: add "*" to beginning of "MoreIdentify"

line 75: sun its/it's

8.	Pager

line 18: maybe change the numbers to variable place-holders?  e.g
[{1stdeskno} {lastdeskno}]  Also, are they _required_?  I don't think so,
because the default ships with only the final number.  

line 78: will defining a default icon for all non-defined apps cause a
problem here?

In the section on config options, is "default is xyz" nicer than "defaults
to"?  (I don't much care, but some style-fascists will.)

line 233: sub yield/yeild.  Also, these 3 lines are opaque.

line 239: Seems to be an example that got left in the config options
section.

line 248: isn't this repeated functionality from feels?  "StubbornIcons"
or something?

line 310: this line wraps; dunno if oyu can do anything about that.
Substitute "{deskno}_background" for "0_background"?

line 337: as line 310

lines 351-2: sub that/this; sub program's/programs

lines 364-5: either "small amounts" or "a small amount"

line 365: "if you are using _a_ small image"

line 386: "ie" trails off.  s/b "i.e."; or, more likely, "e.g."

line 443: sub suit/suite.  Sub user's/users. Add "it" at end of line.

line 455: sub suit/suite.  Sub user's/users.  Del full stop, join w/ next
line.

line 474: sub its/it's


That's it: mostly just typos.  Great job!

Andrew Sullivan | asullivan@sprint.ca (better)| ajsulliv@mcmaster.ca (worse)
                                   *  *  *
                Go to http://lowrent.org/asfaq/, or issue
'finger -l ajsulliv@mcmail.cis.mcmaster.ca' to find the AfterStep FAQ file.

• Next message: "Patch 10"
• Previous message: Ethan "patches 8 & 9"