On Thu, Aug 11, 2011 at 4:20 AM, Vinicius Santos <vinicius.santos.li...@gmail.com> wrote: > Em 10/08/2011 07:34, Dave Page escreveu: >> >> Please see about fixing those issues, and then I can give it a more in >> depth review. > > I fixed the items in question, and performed some more tests. > > Apparently it's OK. > > If any bugs or any suggestions, please let me know.
Looking better :-) - On frmOptions, the combo box to select the refresh mode should be to the right of the label, not underneath. The sizing should following the layout of other tabs. - In "Refresh object on click" mode, if I click a table, and then click the parent database, all nodes below the "Schemas" node are collapsed. - Please use an actual declared enum for the setting, so we can avoid using "magic" numbers in execSelChange() - eg. if (settingRefreshOnClick == REFRESH_OBJECT_ONLY) -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers
