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-08-26 17:44:53 |
Message-ID: | 48B44115.3070609@lelarge.info |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Dave Page a écrit :
> On Wed, Jul 9, 2008 at 12:31 PM, Guillaume Lelarge
> <guillaume(at)lelarge(dot)info> wrote:
> [...]
>> I prefer to add sizing to all (properties') dialogs. This is a work I can
>> do, step by step. I think I should follow these steps in this order:
>>
>> * one patch for each dialog to add the sizing.
>> * one patch to add sizing on common tabs (ie "Variables", "Privileges",
>> and "SQL").
>> * v5 patch for the read-write/read-only mode on sql textfields.
>>
>> OK for these steps?
>
> Yup, sounds like a good plan to me.
>
Back on this. The plan took a lot more time than I initially thought.
Anyways, here is the v5 patch updated.
This patch aims to allow changes on sql text fields. The checkbox need
to be checked to allow typing in the text fields. When checked, other
tabs are unavailable. When un-checked, a confirmation dialog aks the
user if he wants to loose his changes. If yes, it replaces changes with
the previous queries. If no, it keeps the checkbox checked.
It seems to me it's ready for commit.
Comments?
--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com
From | Date | Subject | |
---|---|---|---|
Next Message | Guillaume Lelarge | 2008-08-26 17:48:26 | Re: Enabling SQL text field in the SQL tab of object dialog |
Previous Message | Dave Page | 2008-08-26 14:24:43 | Re: Dialogs Review new patch |