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

Re: Rework on the dialogs UI

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Rework on the dialogs UI
Date: 2011-04-08 09:50:52
Message-ID: 4D9EDA7C.30108@lelarge.info (view raw or flat)
Thread:
Lists: pgadmin-hackers
Le 07/04/2011 17:07, Dave Page a écrit :
> On Thu, Apr 7, 2011 at 2:57 PM, Guillaume Lelarge
> <guillaume(at)lelarge(dot)info> wrote:
>>>> - The catalog properties dialogue crashes horribly.
>>>>
>>>
>>> It doesn't crash on my laptop (I tried with information_schema and
>>> pg_catalog on a PostgreSQL HEAD).
>>>
>>
>> Still didn't reproduce it.
> 
> When it opens I can briefly see the cancel button at the top left
> before it crashes, then:
> 
> Exception Type:  EXC_BAD_ACCESS (SIGBUS)
> Exception Codes: KERN_PROTECTION_FAILURE at 0x00000000000001c8
> Crashed Thread:  0  Dispatch queue: com.apple.main-thread
> 
> Thread 0 Crashed:  Dispatch queue: com.apple.main-thread
> 0   pgAdmin3-Debug                	0x00173f08
> dlgSecurityProperty::OnChangeSize(wxSizeEvent&) + 24
> 1   libwx_base_carbonu-2.8.dylib  	0x014e24d3
> wxEvtHandler::ProcessEventIfMatches(wxEventTableEntryBase const&,
> wxEvtHandler*, wxEvent&) + 131
> 2   libwx_base_carbonu-2.8.dylib  	0x014e2b51
> wxEventHashTable::HandleEvent(wxEvent&, wxEvtHandler*) + 113
> 3   libwx_base_carbonu-2.8.dylib  	0x014e2e7f
> wxEvtHandler::ProcessEvent(wxEvent&) + 207
> 4   libwx_macu_core-2.8.dylib     	0x0102ea3d
> wxTopLevelWindowMac::Show(bool) + 397
> 5   libwx_macu_core-2.8.dylib     	0x00fe1d3d wxDialog::Show(bool) + 45
> 6   pgAdmin3-Debug                	0x0017c47b dlgProperty::Go(bool) + 4155
> 7   pgAdmin3-Debug                	0x0017cd36
> dlgDefaultSecurityProperty::Go(bool, bool, wxString const&, wxString
> const&, wxString const&) + 38
> 8   pgAdmin3-Debug                	0x0019235a dlgSchema::Go(bool) + 474
> 9   pgAdmin3-Debug                	0x0017a37f
> dlgProperty::EditObjectDialog(frmMain*, ctlSQLBox*, pgObject*) + 559
> 10  pgAdmin3-Debug                	0x0017a6f3
> propertyFactory::StartDialog(frmMain*, pgObject*) + 35
> 11  pgAdmin3-Debug                	0x001fb316
> frmMain::OnAction(wxCommandEvent&) + 70
> ...
> ...
> 

I think I know why. The OnChangeSize() method is specific to the Mac
platform. It substracts 550 to the height, but I lower the height a bit,
so 550 is probably too big now.

I tried all this afternoon to have a mac build, but still have no
result. Either you try smaller number in
dlgSecurityProperty::OnChangeSize() or you'll have to wait that I'm able
to compile on my Mac :-/

>>>> - Have you come up with a new set of standard sizes?
>>>>
>>>
>>> Yeah, two sizes:
>>>
>>>  * small one: 220,250
>>
>> new small size is 250,250
> 
> I wonder if it should be 300 wide? The tab labels are all visible now,
> but the edges aren't.
> 

Done in the patch attached.

>>>  * big one: 270,250
>>>
>>> But it seems I forgot dlgServer and dlgTable. Just fixed dlgServer.
>>> Working on dlgTable right now.
>>>
>>
>> Fixed too.
> 
> :-)
> 
>> I fixed also the frmOptions window.
> 
> Not quite - see the attached screenshot.
> 

Should be fixed in the patch attached.

> BTW, that one also seems to be missing the status bar. Wanna fix that
> while you're there?
> 

AFAIR, there is no status bar in the frmOptions window. Do you really
want to add it to this window? what for?


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

Attachment: 0003-Fixes-according-to-Dave-s-answer.patch
Description: text/x-diff (35.5 KB)

In response to

Responses

pgadmin-hackers by date

Next:From: Dave PageDate: 2011-04-08 10:35:11
Subject: Re: Rework on the dialogs UI
Previous:From: Dave PageDate: 2011-04-08 08:42:16
Subject: pgAdmin III commit: Fix capitalisation.

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