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

Re: Buglets in dlgFunction

From: "Dave Page" <dpage(at)pgadmin(dot)org>
To: "Guillaume Lelarge" <guillaume(at)lelarge(dot)info>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Buglets in dlgFunction
Date: 2008-11-14 19:15:20
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgadmin-hackers
On Fri, Nov 14, 2008 at 6:01 PM, Guillaume Lelarge
<guillaume(at)lelarge(dot)info> wrote:
> Guillaume Lelarge a écrit :
>> Dave Page a écrit :
>>> [...]
>>> On Mon, Oct 27, 2008 at 8:23 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>> On Sun, Oct 19, 2008 at 10:33 PM, Guillaume Lelarge
>>>> <guillaume(at)lelarge(dot)info> wrote:
>>>>> Thanks. I still don't like this hack. I hope I'll soon better understand
>>>>> this issue to replace the ugly code with a good one.
>>>> You're going to hate this - but I just tested Gevik's work on Windows,
>>>> and seem to be seeing the SQL and Privilege panes mis-behaving there
>>>> :-(
>>>> I wonder if we just need to defer the restoration of the dialogue size
>>>> to a much later time than it currently is done. Obviously we could
>>>> just do it at the point that you have your hack at the moment, but it
>>>> would be nice to do it in the base class. I tried doing this quickly
>>>> but didn't get too far.
>> I tried it and it failed.
>>>> Putting your existing hack towards the end of
>>>> dlgProperty::Go (but without the WXMAC ifdef) seemed to do the job
>>>> though - albeit still hackily.
>>>> Thoughts?
>> Still a hack but it works. It even works with GTK. The patch attached
>> removes the WXMAC ifdef.
>>>> I also found on Windows that the check boxes for different privileges
>>>> on the privilege panes are touching each other and could use some
>>>> spacing. Can you look at that please?
>> I forgot to add a border to the checkboxes. I tried from 4 to 1. 4 was
>> really big. I still hesitate between 1 et 2. 2 seems better, so that's
>> the value in the patch. I think you should try both of them and tell us
>> which one you prefer.
> Any comments on this? can I apply?

Oh, sorry - somehow it escaped from my TODO list. I'll try to review
later tonight.

Dave Page
EnterpriseDB UK:

In response to

pgadmin-hackers by date

Next:From: Guillaume LelargeDate: 2008-11-14 20:52:56
Subject: Patch for ALTER DATABASE
Previous:From: Guillaume LelargeDate: 2008-11-14 18:01:06
Subject: Re: Buglets in dlgFunction

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