Re: PATCH: Save settings regardless of Find&Replace dialog's close method

From: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
To: "J(dot)F(dot) Oster" <jinfroster(at)mail(dot)ru>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: PATCH: Save settings regardless of Find&Replace dialog's close method
Date: 2014-04-04 06:32:03
Message-ID: CAG7mmox2d--gOkE2fY31=VbGcuCzqQw9PLbhJ=bAMbJOvcZVLQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Checked-in the master.

On Thu, Mar 20, 2014 at 9:03 PM, J.F. Oster <jinfroster(at)mail(dot)ru> wrote:

> Hello,
>
> Make "Find & Replace" dialog in SQL Box remember search settings when
> it is closed (in any way; in fact it's being hidden).
>
> Existing instances of dialog, once invoked again, will keep their own
> settings.
> New instance of dialog will gain settings of the last dialog which has
> been closed before it's invocation.
>
>
>
> >>> By the way, I was puzzled why search dialog's settings are saved
> >>> sometimes and sometimes are not. Depending on dialog's close method:
> >>> 1. Pressing ESC or "Close" form button calls OnCancel() which does not
> >>> save;
> >>> 2. Pressing title [x] button or Alt-F4 (or similar OS hotkey) calls
> >>> OnClose() which does save.
> >>>
> >>> Shouldn't the behaviour be the same regardless of dialog's close
> >>> method? Or at least the difference should be noted in documentation.
>
> DP>> Yes, it should. Patches welcome :-)
>
> JFO> Ok, after the current patch is applied I'll send in another one to
> JFO> make it save settings in both cases.
>
>
> --
> Best regards,
> J.F.
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
>

--
--

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>

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message J.F. Oster 2014-04-04 11:33:21 Re: [pgadmin-support] Search feature request
Previous Message Ashesh Vashi 2014-04-04 06:31:41 pgAdmin III commit: Save the settings of the find dialog even when clos