Skip site navigation (1) Skip section navigation (2)

Re: Increasing size of results grid

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Thom Brown <thom(at)linux(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, Josh Berkus <josh(at)agliodbs(dot)com>, pgadmin-support(at)postgresql(dot)org
Subject: Re: Increasing size of results grid
Date: 2010-10-24 02:13:12
Message-ID: 4CC39638.6000704@lelarge.info (view raw or flat)
Thread:
Lists: pgadmin-support
Le 22/10/2010 04:49, Thom Brown a écrit :
> On 22 October 2010 12:31, Guillaume Lelarge <guillaume(at)lelarge(dot)info> wrote:
>> Le 22/10/2010 03:51, Thom Brown a écrit :
>>> On 22 October 2010 00:34, Guillaume Lelarge <guillaume(at)lelarge(dot)info> wrote:
>>>> Le 19/10/2010 13:29, Thom Brown a écrit :
>>>>> On 19 October 2010 21:24, Thom Brown <thom(at)linux(dot)com> wrote:
>>>>>> On 19 October 2010 21:08, Guillaume Lelarge <guillaume(at)lelarge(dot)info> wrote:
>>>>>>> You're right. Done in this new attached patch.
>>>>>>
>>>>>> Yep, that's excellent :)  One problem is that if you zoom in and
>>>>>> scroll down to a high-numbered row, it's cropped and you can't see the
>>>>>> whole number, so that would have to adjust too.  Might be unreasonable
>>>>>> for me to ask for column widths to also increase proportionately, but
>>>>>> it's better than before with your latest patch.
>>>>>
>>>>> Oh, something else I've noticed.  The zoom in the SQL Editor is the
>>>>> opposite of the grid.  CTRL+scrollwheel up zooms out rather than in.
>>>>>
>>>>
>>>> Both fixed in this new patch. I also fix a BIG issue: my previous patch
>>>> doesn't allow the use of the wheel to move rows :)
>>>>
>>>> So, what about this patch? good enough, I hope :)
>>>
>>> I've tried building it twice because I couldn't see what had changed,
>>> except I notice CTRL+scrollwheel up zooms out now on both the SQL pane
>>> and results. Shouldn't those both be the other way round?  It's just
>>> web browsers all work that way.
>>>
>>
>> I didn't do anything for the SQL pane. In fact, it's a capacity of the
>> component we use. My only change was to make it consistent with the way
>> the integrated component works.
> 
> Okay, fair enough. :)
> 
>>> But I can't see the column widths adjusting, or the row number
>>> container width increasing either.  Running Linux btw.
>>>
>>
>> Row height should change. I had a bug on the column widths stuff. It
>> should now work the wxWidgets way, meaning we only use the labels size
>> to change the width of the columns.
>>
>> New patch attached.
> 
> Yes, row height correctly changed previously.
> 
> Okay, the column width also correctly resizes now.  The row numbers
> still crop though when scaling up.
> 
> As for not changing anything to make it look glossy, not sure what's
> different then.
> 
> Nice work though Guillaume :)
> 

Thanks, commited.

> And I'll continue to be annoying, but would it be easy to make the
> scrollwheel zoom functionality work on Explain, Messages, History and
> even the object browser tree?  (The font size used on my object
> browser always looks too large in Ubuntu).  All a nice-to-have rather
> than a need though.
> 

I created a ticket (#268) for that. No sure we really need this on the
browser tree, not even sure that it can be done.


-- 
Guillaume
 http://www.postgresql.fr
 http://dalibo.com

In response to

pgadmin-support by date

Next:From: Guillaume LelargeDate: 2010-10-24 02:14:32
Subject: Re: Increasing size of results grid
Previous:From: Peter GeogheganDate: 2010-10-23 17:21:25
Subject: Re: Building PgAdmin with a debug build

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group