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

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: Enabling SQL text field in the SQL tab of object dialog
Дата
Msg-id 937d27e10807010817h71a57ba1obe826fcf3a88175a@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Enabling SQL text field in the SQL tab of object dialog  (Guillaume Lelarge <guillaume@lelarge.info>)
Ответы Re: Enabling SQL text field in the SQL tab of object dialog  (Guillaume Lelarge <guillaume@lelarge.info>)
Список 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 по дате отправления:

Предыдущее
От: Guillaume Lelarge
Дата:
Сообщение: Re: Enabling SQL text field in the SQL tab of object dialog
Следующее
От: svn@pgadmin.org
Дата:
Сообщение: SVN Commit by dpage: r7387 - trunk/pgadmin3/pgadmin/ui