[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [lmi] Input-sequence editor testing
From: |
Greg Chicares |
Subject: |
Re: [lmi] Input-sequence editor testing |
Date: |
Sun, 07 Aug 2011 16:25:04 +0000 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.17) Gecko/20110414 Thunderbird/3.1.10 |
On 2011-07-09 23:27Z, Greg Chicares wrote:
> On 2011-07-09 17:32Z, Václav Slavík wrote:
>>
>> On 9 Jul 2011, at 18:29, Greg Chicares wrote:
>>>
>>> (2) In the input-sequence editor, Enter navigates to the next control.
>>> Elsewhere
>>> in lmi, Enter triggers the default pushbutton; can that same style be used
>>> here
>>> for consistency? Inserting 'ok_button_->SetDefault()' near the bottom of
>>> InputSequenceEditor::InputSequenceEditor() seems to work, but I'm not sure
>>> whether something bad would happen if the button is disabled as mentioned
>>> here:
>>
>> No, not at all, that's the right thing to do, sorry for not doing it myself
>> in
>> the first place.
>
> Okay, I'll insert that line right after the buttons are realized:
>
> buttons->Realize();
> + ok_button_->SetDefault();
>
> top->Add(buttons, wxSizerFlags().Expand().Border());
Committed 20110807T1618Z, revision 5233.