Re: Enabling SQL text field in the SQL tab of object dialog

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Dave Page <dpage(at)postgresql(dot)org>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Enabling SQL text field in the SQL tab of object dialog
Date: 2008-07-09 08:18:25
Message-ID: 48747451.6040607@lelarge.info
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Dave Page a écrit :
> (apologies for not reviewing the latest patch yet - we moved offices
> on Monday so I've been tied up with that for the last few days)
>

No problem, don't worry :)

> On Wed, Jul 9, 2008 at 12:07 AM, Guillaume Lelarge
> <guillaume(at)lelarge(dot)info> wrote:
>
>> I worked on the remaining issue tonight and it seems I can fix it this way:
>> I add a wxFlexGridSizer and a few sizers on the .xrc file. It works well on
>> the Linux and on the Win32 plateforms (I only check with dlgDatabase.xrc
>> file). If I'm right, this means I need to add these widgets on each
>> properties' dialog that doesn't already have them (all but functions and
>> trigger' ones).
>
> I don't understand what you mean. The SQL tab is programmatically
> added to each dialogue, so what do you propose to add the sizer to in
> the XRC files?
>

The last patch I sent works on GTK+. When I try it on Windows, it works
for dlgFunction and dlgTrigger and doesn't with all other dlg*. I tried
to understand what could be wrong with dlgDatabase. dlgFunction already
has a wxFlexGridSizer on the .xrc file. The wxFlexGridSizer contains the
notebook. So, what I did is adding another wxFlexGridSizer in
dlgDatabase.xrc and a few other widgets to make it look like the
dlgFunction.xrc file. See the patch attached. This is a quick and dirty
patch... it adds Apply and Help buttons that are not handled, it also
adds a style tag (for the wxDialog) which is not necessary. Anyways,
with this new component on the xrc file, it works on Windows.

> Besides, anything that is added via an XRC file should be possible
> through C++ (the XRC files are translated to C++ in fact), so we
> shouldn't need to hack any XRC files and should able to continue with
> the current implementation in the base class.
>
> It seems to me that this is taking far too long to solve what should
> be a simple problem. Where are we going wrong? I need to try to find
> some time to look at this in more depth.
>

If you could, it would be great. But don't hurry, I can wait till you
have more time :)

Thanks.

--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com

Attachment Content-Type Size
dlgDatabase.xrc.patch text/x-patch 3.1 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2008-07-09 10:54:42 Re: Enabling SQL text field in the SQL tab of object dialog
Previous Message Dave Page 2008-07-09 07:41:16 Re: Enabling SQL text field in the SQL tab of object dialog