Please do not reply to this email- if you want to comment on the bug, go to the URL shown below and enter your comments there.
Changed by [EMAIL PROTECTED] http://bugzilla.ximian.com/show_bug.cgi?id=57748 --- shadow/57748 2005-07-20 12:39:43.000000000 -0400 +++ shadow/57748.tmp.27621 2005-08-05 19:00:22.000000000 -0400 @@ -57,6 +57,19 @@ ------- Additional Comments From [EMAIL PROTECTED] 2005-07-20 12:39 ------- You had lots of extra diffs in the glade file. Also, lots of your code uses spaces rather than \t which makes the indentation look funny. IMHO, the about dialog doesn't need to be moved out of browser.cs, but thats a decision for miggy + +------- Additional Comments From [EMAIL PROTECTED] 2005-08-05 19:00 ------- +Ben, + +I've cleaned up AboutDialog.cs and made it follow the mono coding +guidelines. About the glade diff, the changes are significant because +I completely removed the old about dialog and created a new one from +scratch. +With regards to whether or not to have the about dialog in a separate +file or not, I think browser.cs is getting ugly as it is and it made +more sense to me at the time to split it up. + + _______________________________________________ mono-bugs maillist - [email protected] http://lists.ximian.com/mailman/listinfo/mono-bugs
