Re: [pgadmin-hackers] PATCH: save and restore the settings of the search objects dialog

2014-07-09 Thread Dave Page
Thanks - patch applied. On Tue, Jul 8, 2014 at 7:28 PM, Dmitriy Olshevskiy wrote: > Hello J.F. > Honestly, I don't sure that SaveSettings() must be called every time you > push the Find button. > What about the destructor method? (see the attached file) > > 08.07.2014 21:16, J.F. Oster пишет: > >

Re: [pgadmin-hackers] PATCH: save and restore the settings of the search objects dialog

2014-07-08 Thread Dmitriy Olshevskiy
Hello J.F. Honestly, I don't sure that SaveSettings() must be called every time you push the Find button. What about the destructor method? (see the attached file) 08.07.2014 21:16, J.F. Oster пишет: Hello, On Sun, Jun 22, 2014 at 11:29 AM, Dmitriy Olshevskiy wrote: Hello. I think it'll be

Re: [pgadmin-hackers] PATCH: save and restore the settings of the search objects dialog

2014-07-08 Thread J.F. Oster
Hello, On Sun, Jun 22, 2014 at 11:29 AM, Dmitriy Olshevskiy wrote: > Hello. > I think it'll be useful to save the settings of search objects dialog. So I > decided to add this feature. > Please see the attached patch. > Thank you. Reporting my user experience :) SaveSettings() is called on Cance

Re: [pgadmin-hackers] PATCH: save and restore the settings of the search objects dialog

2014-06-27 Thread Dave Page
Thanks, patch committed. On Sun, Jun 22, 2014 at 11:29 AM, Dmitriy Olshevskiy wrote: > Hello. > I think it'll be useful to save the settings of search objects dialog. So I > decided to add this feature. > Please see the attached patch. > Thank you. > > -- > Dmitriy Olshevskiy > -- Dave Page B

[pgadmin-hackers] PATCH: save and restore the settings of the search objects dialog

2014-06-22 Thread Dmitriy Olshevskiy
Hello. I think it'll be useful to save the settings of search objects dialog. So I decided to add this feature. Please see the attached patch. Thank you. -- Dmitriy Olshevskiy From 904ba63af8d182ea8f5a00a6d45c3604d21eb92b Mon Sep 17 00:00:00 2001 From: olshevskiy87 Date: Sun, 22 Jun 2014 13:5