[Mono-docs-list] Submitted patches to 38490

Lee Mallabone mono-docs@fonicmonkey.net
23 Feb 2003 21:00:54 -0500


On Fri, 2003-02-21 at 14:52, Kevin Breit wrote:
> Hey,
> 	Could people please review:
> http://bugzilla.ximian.com/showattachment.cgi?attach_id=3421
[which is Gtk.Arrow]

I hope I'm not stepping on any toes, as I know it's Duncan that has
final say, but...

The Set() method got removed on cvs, as ArrowType and ShadowType can be
set with properties.

Other than that, I'd say mark up any enums as xml. For example, instead
of:
"The style of the arrow can be one of the following: None, In, Out"

It's a better idea to replace 'None' with something like: 
<see cref="P:Gtk.ShadowType.None"/>

It might sound like a nitpick, but I personally think consistent links
like that will make the API docs as a whole so much nicer to browse...

Regards,

Lee.