[PATCH setup] Make Enter in the user URL box cause ADD instead of NEXT
Fri Dec 8 16:14:00 GMT 2017
On 12/8/2017 10:48 AM, Jon Turney wrote:
> On 07/12/2017 20:46, Ken Brown wrote:
>> On 12/7/2017 1:35 PM, Jon Turney wrote:
>>> On 05/12/2017 17:32, Ken Brown wrote:
>>>> On 12/5/2017 11:03 AM, Ken Brown wrote:
>>>>> On 12/5/2017 7:58 AM, Jon Turney wrote:
>>>>>> The search textbox on the package chooser page needs the same fix.
>>>>> It's not immediately clear to me how to do this, since I don't know
>>>>> what the default pushbutton should be while the user is typing in
>>>>> the search box.
>>>>> One possibility is to convert the label "Search" to the left of the
>>>>> box to a SEARCH pushbutton, whose effect is to call
>>>>> OnTimerMessage(). If we make this the default, then pressing Enter
>>>>> will cause the search filter to immediately take effect, which is
>>>>> probably what the user expects.
>>> It seems a bit weird to have a button which automatically pushes
>>> itself half a second after you finish typing.
>>> Attached is my attempt, which (ab)uses an invisible button.
>> I agree, this is better than my version.
>>>> Something like the attached?Â This might not be quite right, because
>>>> the previous default button is never restored.Â I'm not sure how
>>>> important that is.
>>> I think it's something that should be done, if possible, so I added
>> In my testing, 'Next' does indeed become the default button after I
>> click outside of the textbox, but there's no visual indication of this.
> This is interesting: if you use TAB to move the focus out of the
> textbox, then first "Clear" gets highlight (because it's a pushbutton
> and enter pushes it), TAB again and "Current is selected (but "Next"
> gets the highlight, because that's what enter pushes)
> If you click to move the focus, it only seems to update the highlight
> the second time you do that.
> Which I guess suggests we should be ensuring the highlight is drawn on
> But once I do that, it seems I need to explicitly remove as well, which
> gives the attached, incremental patch.
That seems to fix it. Thanks.
More information about the Cygwin-apps