Re: Enabling SQL text field in the SQL tab of object dialog - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Enabling SQL text field in the SQL tab of object dialog
Date
Msg-id 937d27e10807010817h71a57ba1obe826fcf3a88175a@mail.gmail.com
Whole thread Raw
In response to Re: Enabling SQL text field in the SQL tab of object dialog  (Guillaume Lelarge <guillaume@lelarge.info>)
Responses Re: Enabling SQL text field in the SQL tab of object dialog  (Guillaume Lelarge <guillaume@lelarge.info>)
List pgadmin-hackers
On Tue, Jul 1, 2008 at 11:52 AM, Guillaume Lelarge
<guillaume@lelarge.info> wrote:

> I'm at the PostgreSQL booth, for the RMLL in Mont-de-Marsan
> (http://rmll.info). I'm trying the wifi system :)

:-)

> Here is the patch that fixes your issue on Win32. Works great on Win32 and
> Linux... I'm not sure for Mac OS X though.

Hmm, this still isn't looking so good I'm afraid:

- If I open the properties dialogue on a database, click the SQL tab
and then uncheck Read only, having changed nothing, I get asked if I
want to cancel my edit. I believe this is because in the edit case,
GetSQL() will return nothing, whilst the textbox may contain '--
nothing to change'

- The sizing is still wrong on Mac. Worse, the text boxes no longer
resize on the resizeable dialogues (dlgFunction, dlgPackage,
dlgTrigger and friends).

I would suggest  avoiding trying to calculate the dimensions in code -
in my experience this almost never works well. Instead, I would
suggest laying the two textboxes and the checkbox in a sizer and
letting that do the work. Not only should that be easier to code (once
you fully understand sizers!), but it should work properly on all
platforms, and be resize-friendly.

The same should probably be done for other standard tabs (eg,
privileges, gucs) when you update all the XRC files as you discussed
previously.

Sorry, I know this isn't what you wanted to hear - but we'll get there :-)

--
Dave Page
EnterpriseDB UK: http://www.enterprisedb.com

pgadmin-hackers by date:

Previous
From: Guillaume Lelarge
Date:
Subject: Re: Enabling SQL text field in the SQL tab of object dialog
Next
From: svn@pgadmin.org
Date:
Subject: SVN Commit by dpage: r7387 - trunk/pgadmin3/pgadmin/ui