From: | Michael Shapiro <mshapiro51(at)gmail(dot)com> |
---|---|
To: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
Cc: | Jeff Adams <jadams(at)azavea(dot)com>, Peter Geoghegan <peter(dot)geoghegan86(at)gmail(dot)com>, pgadmin-support(at)postgresql(dot)org |
Subject: | Re: Bug with exceptionally long values. |
Date: | 2010-07-28 13:28:24 |
Message-ID: | AANLkTin5hTxKcxdqiyvYM+jajG7=bT70zdSwTMP-YQck@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-support |
I think changing the background but leaving the current value "as is" is
better than changing the value. That way you indicate that the value is
there (and can be retrieved with copy/paste) but that you can't show it (and
perhaps add a pop-up hint when the mouse is moved over the cell)
On Wed, Jul 28, 2010 at 8:19 AM, Guillaume Lelarge
<guillaume(at)lelarge(dot)info>wrote:
> Le 28/07/2010 13:59, Jeff Adams a écrit :
> > The class that I experience the problem with is the ctlSQLGrid, which
> > extends wxGrid. I don't know wxWidgets at all, but in the grid sample
> class
> > they have an example of setting a cell renderer, perhaps you could add a
> > cell renderer that changes the background color [or even renders a string
> > that says "Sorry, too long!"] based on the strlen? That should have
> fairly
> > minimal performance impact since it will only affect the N rows being
> > actually rendered, not the entire result set.
> >
> > The example is /wxWidgets-2.8.11/samples/grid/griddemo.cpp line 299.
> >
>
> Well, I think it would hurt the performance. But, we should check that.
> And it would be a pretty good way to show a NULL value.
>
>
> --
> Guillaume
> http://www.postgresql.fr
> http://dalibo.com
>
From | Date | Subject | |
---|---|---|---|
Next Message | Little, Douglas | 2010-07-28 13:44:03 | Re: It takes a long time for pgAdmin to retrieve database schema details |
Previous Message | Guillaume Lelarge | 2010-07-28 13:19:02 | Re: Bug with exceptionally long values. |