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. _______________________________________________ Mono-docs-list maillist - [EMAIL PROTECTED] http://lists.ximian.com/mailman/listinfo/mono-docs-list
