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 AANLkTikvMyUrgx_MNMP39rqmPh5tYKAgmbcQ915YsHt=@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
Re: Fwd: Pgadmin 1.12 rc1 editing bug
List pgadmin-hackers
On Fri, Sep 3, 2010 at 11:39 PM, Guillaume Lelarge
<guillaume@lelarge.info> wrote:
> Le 03/09/2010 23:07, Dave Page a écrit :
>> On Fri, Sep 3, 2010 at 9:24 PM, Steffen Kuhn <pg@kuhnsteffen.de> wrote:
>>>  Hi Guillaume,
>>>
>>> the delete problem is following:
>>> deleting rows has got the shortcut 'del'. Disabeling row deletion disables
>>> 'del'.
>>> Even the key event WXK_DELETE is not fired.
>>> So the solution of this problem which should been there since 'del' is the
>>> shortcut for row delete
>>> is to change the shortcut to eg. 'ctrl-del', to get the default behavior of
>>> the del key.
>>> Could you please comment this.
>>
>> Del on its own is the correct key. It should be able to do the "right
>> thing" based on whether or not a cell is in edit mode.
>>
>
> Yeah, I agree. See patch attached (not yet commited).

Hmm, that seems to be setting up the button to handle deletion of
text, which is not really consistent with anything else. Can we leave
the button to be enabled/disabled as it currently is, and ensure the
right thing is done in edit more? That would also avoid breaking a
translation...

--
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: Ljubomir Erdeljan
Date:
Subject: foreign key problem resolved
Next
From: "Steffen Kuhn"
Date:
Subject: Re: Fwd: Pgadmin 1.12 rc1 editing bug