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

From Steffen Kuhn
Subject Re: Fwd: Pgadmin 1.12 rc1 editing bug
Date
Msg-id 4C815979.1080400@kuhnsteffen.de
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
  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.

Working on this I located other issues in the edit context:
_Select some text use context menu an try to perform an action.
Nothing happens, because as I assume selection and focus is lost while
opening the conext menu.
_Select a numeric value and use context menu. A totally different
context menu occurs (system context menu includeing unicode input
features, icons etc.),
which I know from situations in which no menu was defined.

For the shortcut issue I attach a patch.
The new rest will take some time, because for newbie eyes this looks not
trivial.

Regards Steffen

On 03.09.2010 17:12, Guillaume Lelarge wrote:
> Le 03/09/2010 17:08, Steffen Kuhn a écrit :
>>   Okay I'll do my best. ;-)
>>
> Thanks. That gives me more time for the GSoC. I really appreciate that.
>
>> Did I get you right to just perform a delete for to current text
>> selection or the letter after the cursor
> Yes.
>
>> and not enabling menu and toolbar (commented with 'Delete selected
>> rows.') ?
>>
> The icon on the toolbar is not at fault here. It's a delete row button,
> and it should stay that way.
>
> The Delete item in the Edit menu should be enabled when an editor is
> open. And disabled otherwise.
>
>


Attachment

pgadmin-hackers by date:

Previous
From: Guillaume Lelarge
Date:
Subject: Re: Fwd: Pgadmin 1.12 rc1 editing bug
Next
From: Dave Page
Date:
Subject: Re: Fwd: Pgadmin 1.12 rc1 editing bug