Looks good to me. Checked-in! --
Thanks & Regards, Ashesh Vashi EnterpriseDB INDIA: Enterprise PostgreSQL Company <http://www.enterprisedb.com> *http://www.linkedin.com/in/asheshvashi* <http://www.linkedin.com/in/asheshvashi> On Wed, Jul 16, 2014 at 1:37 PM, Dave Page <dp...@pgadmin.org> wrote: > Ashesh - can you look at this ASAP please, and commit if it looks OK? > > > On Sat, Jul 12, 2014 at 12:06 PM, Dmitriy Olshevskiy <olshevski...@bk.ru> > wrote: > >> Hello. >> >> Please see this patch. These are changes in the macros manage dialog: >> - the "Save" button is enabled after any change in the query window >> - RestorePosition() in the constructor >> - unnecessary variable appQueryModify has been removed >> - changed the condition that checks whether the title and query were >> filled (method SetMacro()) >> >> Thank you. >> >> -- >> Dmitriy Olshevskiy >> >> > > > -- > Dave Page > Blog: http://pgsnake.blogspot.com > Twitter: @pgsnake > > EnterpriseDB UK: http://www.enterprisedb.com > The Enterprise PostgreSQL Company >