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. Jeff On Tue, Jul 27, 2010 at 4:39 PM, Guillaume Lelarge <guilla...@lelarge.info>wrote: > Le 27/07/2010 22:26, Jeff Adams a écrit : > > The bug is definitely a wxWidgets bug, and exists both in 2.8.11 and > 2.9.1. > > I was able to reproduce it in their "grid" sample. > > > > I submitted a bug to the wxWidgets project: > > > > http://trac.wxwidgets.org/ticket/12276 > > > > Do you think pgAdmin should implement some sort of temporary fix in the > > meanwhile? > > > > It would be best to have a (even temporary) fix. I'm not sure how we can > do that. Having a special background colour could do that. But that > means we need to check the length of every cells. I suppose that, if you > have cells with more than 50k characters, you'll also have big tables. > It sure will take a lot of time to check that. And what should we do if > we find cells with more than X thousand characters? Highlight them? Draw > something? I don't know. All I can say is that it won't be cheap > (performance-wise). > > > -- > Guillaume > http://www.postgresql.fr > http://dalibo.com > -- Jeff Adams Azavea (formerly Avencia) | T: 215.701.7717 | F: 215.925.2663 www.azavea.com
