Re: Fwd: Pgadmin 1.12 rc1 editing bug

Le 06/09/2010 21:11, Dave Page a écrit :
> On Mon, Sep 6, 2010 at 5:44 PM, Guillaume Lelarge
> <guillaume(at)lelarge(dot)info> wrote:
>> Le 06/09/2010 18:37, Dave Page a écrit :
>>> On Mon, Sep 6, 2010 at 5:34 PM, Guillaume Lelarge
>>> <guillaume(at)lelarge(dot)info> wrote:
>>>> Le 06/09/2010 18:13, Dave Page a écrit :
>>>>> On Mon, Sep 6, 2010 at 5:03 PM, Guillaume Lelarge
>>>>> <guillaume(at)lelarge(dot)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.
>
>

OK, then. Commited on 1.12 and master. Thanks.

--
Guillaume
http://www.postgresql.fr
http://dalibo.com