Re: Query results in grid, new patch - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Query results in grid, new patch
Date
Msg-id E7F85A1B5FF8D44C8A1AF6885BC9A0E40103E494@ratbert.vale-housing.co.uk
Whole thread Raw
In response to Query results in grid, new patch  ("Edward Di Geronimo Jr." <edigeronimo@xtracards.com>)
Responses Re: Query results in grid, new patch
Re: Query results in grid, new patch
List pgadmin-hackers

> -----Original Message-----
> From: Edward Di Geronimo Jr. [mailto:edigeronimo@xtracards.com]
> Sent: 27 February 2006 18:39
> To: Dave Page
> Cc: Edward Di Geronimo Jr.; pgadmin-hackers@postgresql.org
> Subject: RE: [pgadmin-hackers] Query results in grid, new patch
>
> Hmmm, it does seem better now than it used to. Still one problem I can
> find. Run a query. Click somewhere in the edit text box.
> Click on a column
> or row header to select an entire row. The focus stays in the
> edit text
> box. If you click on a cell first then select the row or
> column it works
> ok, but it would be nicer if this worked.

OK, can this be solved with wx ctlGrid class derived from wxGrid that
does a SetFocus on itself when a column or row is selected?

> > Other than that, my only comment is the column headers
> which look even
> > messier than they did previously (because the text appears to be
> > centered rather than left justified). Can you alter it to
> place the data
> > type on the line below, as per the Edit Grid please?
>
> I changed it to left justify the text and display the data type on the
> line below.
>
> New patch attached.

Thanks, looking good. We should probably tweak the edit grid to match
the left justified style when this gets accepted.

Note to Andreas: unless you have any concrete technical reasons why this
patch should not be applied, I'm fully intending to accept it once the
current wrinkles are ironed out. Please shout now rather than later!!

Regards, Dave.

pgadmin-hackers by date:

Previous
From: OpenMacNews
Date:
Subject: Re: Buildproblems on OSX 10.3 due to libxml2
Next
From: "Edward Di Geronimo Jr."
Date:
Subject: Re: Query results in grid, new patch