Le 16/03/2011 18:51, Nick Treleaven a écrit : > On Wed, 16 Mar 2011 10:43:08 -0700 > Matthew Brush <[email protected]> wrote: > >> I fixed up the layout and alignment of widgets in the class builder >> plugin. I also fixed a small bug which was causing an assertion warning. >> >> There's 3 patches: >> >> #0001 >> The UI cleanup using a GtkTable, for side-by-side before and after >> screenshots, see [1][2][3] > ... >> [1] http://codebrainz.ca/images/classbuilder/c++_before_after.png >> [2] http://codebrainz.ca/images/classbuilder/gtk+_before_after.png >> [3] http://codebrainz.ca/images/classbuilder/php_before_after.png > > My comments: > Changing 'Foo Bar:' to 'Foo bar:' in labels is good.
Not sure, isn't "Foo Bar" the classic style in Geany? > Increasing horizontal separation between radio buttons - not sure, but > perhaps more usable. > Unaligning entry fields seems bad. > Removing filetype from dialog title (even though each dialog has > different fields) seems bad. > Increasing vertical separations seems bad (out of keeping with Geany > etc). Actually the screenshots seems inverted: the new is on the left, old on the right (as far as I can tell from the current in my Geany version -- though I thought the fields were aligned...). Cheers, Colomban _______________________________________________ Geany-devel mailing list [email protected] http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
