Re: "Clear window" in the SQL Editor

Поиск
Список
Период
Сортировка
От Guillaume Lelarge
Тема Re: "Clear window" in the SQL Editor
Дата
Msg-id 4D0365A6.7090905@lelarge.info
обсуждение исходный текст
Ответ на Re: "Clear window" in the SQL Editor  (Guillaume Lelarge <guillaume@lelarge.info>)
Ответы Re: "Clear window" in the SQL Editor  (Vinicius Santos <vinicius.santos.lista@gmail.com>)
Список 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

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: "pgAdmin Trac"
Дата:
Сообщение: [pgAdmin III] #292: New syntax for creating/altering an operator class
Следующее
От: Dave Page
Дата:
Сообщение: Re: New minor release?