Re: [pgAdmin4][Patch][Feature_1535]: Pressing ESC from within a dialog box should act like "Cancel" button

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Surinder Kumar <surinder(dot)kumar(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch][Feature_1535]: Pressing ESC from within a dialog box should act like "Cancel" button
Date: 2017-06-12 11:50:15
Message-ID: CA+OCxoxkvEzUQujBiMnBQ03iLgxYQQxyBqYCvJEMB=aYwdAM_w@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Could you rebase this please?

Thanks.

On Fri, Jun 9, 2017 at 10:40 AM, Surinder Kumar
<surinder(dot)kumar(at)enterprisedb(dot)com> wrote:
> Ignore previous patch as it doesn't contains new changes.
> Please find attached patch.
>
>
> On Fri, Jun 9, 2017 at 3:06 PM, Surinder Kumar
> <surinder(dot)kumar(at)enterprisedb(dot)com> wrote:
>>
>> Hi
>>
>> PFA patch and review.
>>
>>
>> On Fri, Jun 9, 2017 at 1:47 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>
>>> Hi
>>>
>>> Ahh, now we have a bug. In the properties dialogues, if you edit a
>>> comment, as soon as the Save button gets enabled, hitting Return will
>>> Save the changes instead of adding a newline to the comment. That's
>>> clearly not right :-(
>>
>> Now if event.target is its child elements(for example input, textarea
>> etc), the function "handleKeyDown(event)" won't call.
>>>
>>>
>>> On Fri, Jun 9, 2017 at 7:48 AM, Surinder Kumar
>>> <surinder(dot)kumar(at)enterprisedb(dot)com> wrote:
>>> > Hi
>>> >
>>> > Please find updated patch.
>>> >
>>> > Thanks,
>>> > Surinder
>>> >
>>> > On Thu, Jun 8, 2017 at 7:16 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>> >>
>>> >> Hi
>>> >>
>>> >> On Thu, Jun 8, 2017 at 5:38 AM, Surinder Kumar
>>> >> <surinder(dot)kumar(at)enterprisedb(dot)com> wrote:
>>> >> > Hi
>>> >> >
>>> >> > The attribute key for Ok, Save buttons should be mapped to
>>> >> > 13(Enter),
>>> >> > for Help - 112(F1) and
>>> >> > for Cancel - 27(Esc)
>>> >> > The above changes are made into all dialog related template files.
>>> >> >
>>> >> > PFA patch and review.
>>> >>
>>> >> Looks good. Just a couple of thoughts:
>>> >>
>>> >> - You've missed the object property dialogues. This really needs to be
>>> >> consistent across all dialogues :-)
>>> >>
>>> >> - The behaviour with Enter is a bit odd, because priority will be
>>> >> given the the focussed control to handle it. That's what we want if
>>> >> it's a text area for example, but on the other hand, if you open the
>>> >> Preference dialogue, then the Help button gets focus first, so Enter
>>> >> there will end up opening the docs. I suspect in most cases the
>>> >> behaviour you've got is fine, so I'm not inclined to change it. <plays
>>> >> a little> Hmm, I see similar behaviour in some other apps. Let's leave
>>> >> it as you've done it :-)
>>> >>
>>> >> Please add support for the properties dialogues, then I think we're
>>> >> good
>>> >> to go.
>>> >
>>> > Done.
>>> >>
>>> >>
>>> >> Thanks!
>>> >>
>>> >> --
>>> >> Dave Page
>>> >> Blog: http://pgsnake.blogspot.com
>>> >> Twitter: @pgsnake
>>> >>
>>> >> EnterpriseDB UK: http://www.enterprisedb.com
>>> >> The Enterprise PostgreSQL Company
>>> >
>>> >
>>>
>>>
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>
>>
>

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2017-06-12 11:53:12 pgAdmin 4 commit: Fix error on the stats tab with PG10. Also, rename th
Previous Message Dave Page 2017-06-12 11:49:26 Re: [pgAdmin4][Patch]: Fixed RM 2324 - PostGIS datatypes not showing up properly on SQL tab.