On Fri, Nov 14, 2008 at 6:01 PM, Guillaume Lelarge
<guillaume@lelarge.info> wrote:
> Guillaume Lelarge a écrit :
>> Dave Page a écrit :
>>> [...]
>>> On Mon, Oct 27, 2008 at 8:23 AM, Dave Page <dpage@pgadmin.org> wrote:
>>>> On Sun, Oct 19, 2008 at 10:33 PM, Guillaume Lelarge
>>>> <guillaume@lelarge.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: http://www.enterprisedb.com