Re: Lightspeed for frmQuery and other issues.

From: "Dave Page" <dpage(at)vale-housing(dot)co(dot)uk>
To: <pgadmin(at)pse-consulting(dot)de>, <dpage(at)vale-housing(dot)co(dot)uk>
Cc: <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Lightspeed for frmQuery and other issues.
Date: 2006-04-30 14:04:08
Message-ID: 007e01c66c5e$fb04c666$6a01a8c0@valehousing.co.uk
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

-----Original Message-----
From: "Andreas Pflug"<pgadmin(at)pse-consulting(dot)de>
Sent: 30/04/06 14:48:08
To: "Dave Page"<dpage(at)vale-housing(dot)co(dot)uk>
Cc: "pgadmin-hackers(at)postgresql(dot)org"<pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Lightspeed for frmQuery and other issues.

> EditGrid is untouched.
> copying of arbitrary groups of cells did *never* work correct. The
> current implementation will omit cells if it doesn't know how to handle
> it. This is not acceptable: The ctl needs to be rewritten to not accept
> selections that can't be executed.

I've never run into such a bug, nor has it been reported. How can it be reproduced? Had it been reported, it would have been looked at of course.

> As I already may have mentioned earlier, I don't intend to spent time on
> the crappy wxGrid. So if anybody else likes to do so, he/she should pay
> attention not to change the ctlSQLResult interface for that.

With or without a bug, removing useful functionality is completely unnacceptable. You would be understandably pissed off if a feature you used was removed, and I would ensure the author reimplemented it, or rollback the change. Bug aside, please fix the copy to clipboard to match what was there.

/D

-----Unmodified Original Message-----
Dave Page wrote:
>
> -----Original Message----- From: "Andreas
> Pflug"<pgadmin(at)pse-consulting(dot)de> Sent: 30/04/06 12:24:08 To: "Dave
> Page"<dpage(at)vale-housing(dot)co(dot)uk> Cc:
> "pgadmin-hackers(at)postgresql(dot)org"<pgadmin-hackers(at)postgresql(dot)org>
> Subject: Re: Lightspeed for frmQuery and other issues.
>
>
>>> Cool, nice work. I assume all the clipboard stuff still works?
>>
>> Depends on what you call "the clipboard stuff". Everything that
>> worked in 1.4 still works, i.e. single/multiple row copy. To select
>> columns, there's a special syntax to the SELECT command....
>
>
> Then you have some work to finish - both the edit and query grid
> could copy individual cells, rows, columns or arbitrary groups of
> cells to the clipboard - functionality that I for one am already
> using on a regular basis.

EditGrid is untouched.
copying of arbitrary groups of cells did *never* work correct. The
current implementation will omit cells if it doesn't know how to handle
it. This is not acceptable: The ctl needs to be rewritten to not accept
selections that can't be executed.
As I already may have mentioned earlier, I don't intend to spent time on
the crappy wxGrid. So if anybody else likes to do so, he/she should pay
attention not to change the ctlSQLResult interface for that.

Regards,
Andreas

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message svn 2006-04-30 14:27:25 SVN Commit by andreas: r5103 - trunk/pgadmin3
Previous Message Andreas Pflug 2006-04-30 13:48:03 Re: Lightspeed for frmQuery and other issues.