Re: Fwd: Pgadmin 1.12 rc1 editing bug - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Fwd: Pgadmin 1.12 rc1 editing bug
Date
Msg-id AANLkTikDrPJV9CzRQKX_L09MzbcGWN+HE6KTT3Av00OB@mail.gmail.com
Whole thread Raw
In response to Re: Fwd: Pgadmin 1.12 rc1 editing bug  (Guillaume Lelarge <guillaume@lelarge.info>)
Responses Re: Fwd: Pgadmin 1.12 rc1 editing bug
List pgadmin-hackers
On Mon, Sep 6, 2010 at 5:44 PM, Guillaume Lelarge
<guillaume@lelarge.info> wrote:
> Le 06/09/2010 18:37, Dave Page a écrit :
>> On Mon, Sep 6, 2010 at 5:34 PM, Guillaume Lelarge
>> <guillaume@lelarge.info> wrote:
>>> Le 06/09/2010 18:13, Dave Page a écrit :
>>>> On Mon, Sep 6, 2010 at 5:03 PM, Guillaume Lelarge
>>>> <guillaume@lelarge.info> wrote:
>>>>> It's possible to disable the toolbar button, but we'll need to keep the
>>>>> menu item enabled so that the DEL key still triggers the OnDelete()
>>>>> method. Is this what you want?
>>>>
>>>> What about adding an accelerator for the DEL key binding, so it
>>>> doesn't matter if the menu is disabled or not?
>>>>
>>>
>>> There's already an accelerator entry for the DEL key. If the menu is
>>> disabled, the method is not fired. So, I only know two solutions: don't
>>> disable the menu item, don't show the Del keybinding in the menu item.
>>
>> Does creating the accelerator table before the menu make any difference?
>>
>
> Nope, doesn't work.

I say we remove the shortcut from the menu option then.


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

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

pgadmin-hackers by date:

Previous
From: Guillaume Lelarge
Date:
Subject: Re: Fwd: Pgadmin 1.12 rc1 editing bug
Next
From: Guillaume Lelarge
Date:
Subject: pgAdmin III commit: Make sure the DEL key is usable in the frmEditGrid