Hi Brian,

On Thu, 2008-07-31 at 16:31 -0500, Brian Cameron wrote:

> Jedy:
> 
> > SYNOPSIS
> >      evolution-webcal [--help ]     [--display=DISPLAY] [uri]
> 
> The [uri] should be documented in an OPERANDS section.  I didn't
> fix this, can you correct?

Yes, I will.


> 
> You also failed to list the --version option, which I fixed.  Also
> "DISPLAY" should be just "display".  I also fixed this.
> 
> > DESCRIPTION
> >      evolution-webcal is the default handler for all Webcal  URI.
> >      When   you       click   a  URI  which  begins  with  webcal://,
> >      evolution-webcal will be called. And if the URI  is  correct
> >      it     will create the calendar in Evolution.
> 
> The above had some grammar issues, which I corrected.  Please review
> my changes.
> 
> When you say "create the calendar in Evolution" do you mean to say
> "will create a calendar event in Evolution"?  Perhaps this could be
> made a bit more clear?

No, I do not mean a calendar event. "create the calendar folder in
Evolution" may be better.



> [...]
> 
> > SEE ALSO
> >      evolution(1), attributes(5),
> 
> I removed the comma after attributes(5).  Are there any other
> evolution related manpages that should be referenced?

I will check, but I think there 2 should be enough.

Regards,

Jedy

> 
> Brian
> 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: 
<http://mail.opensolaris.org/pipermail/jds-review/attachments/20080802/11fce59e/attachment.html>

Reply via email to