Re: "Clear window" in the SQL Editor

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Vinicius Santos <vinicius(dot)santos(dot)lista(at)gmail(dot)com>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: "Clear window" in the SQL Editor
Date: 2010-12-11 11:51:02
Message-ID: 4D0365A6.7090905@lelarge.info
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Le 06/12/2010 12:34, Guillaume Lelarge a écrit :
> Le 29/11/2010 23:35, Vinicius Santos a écrit :
>> [...]
>> In the SQL editor window, when the focus is outside of any text box, the
>> button "clear window" does nothing.
>>
>> I believe that confuse the user. In this case the button should clear
>> the SQL editor.
>>
>> Do you agree?
>>
>> I made this change.
>>
>> void frmQuery::OnClear(wxCommandEvent& ev)
>> {
>> wxWindow *wnd=currentControl();
>>
>> if (wnd == sqlQuery)
>> sqlQuery->ClearAll();
>> else if (wnd == msgResult)
>> msgResult->Clear();
>> else if (wnd == msgHistory)
>> msgHistory->Clear();
>> else if (wnd == scratchPad)
>> scratchPad->Clear();
>> + else
>> + sqlQuery->ClearAll();
>> }
>>
>>
>> Sorry my bad English. I am Brazilian.
>> The Google translator is helping me.
>>
>
> Sorry for not answering sooner, it seems a good change to me, I need to
> check it first.
>

OK, after some thinking, it would be best to disable the "Clear window"
items rather than clearing the query. The same needs to be done for Cut,
Copy, Paste items.

Do you want to work on such a patch?

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

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2010-12-11 13:59:09 Re: New minor release?
Previous Message pgAdmin Trac 2010-12-11 11:36:16 [pgAdmin III] #292: New syntax for creating/altering an operator class