Re: [PATCH setup] Make Enter in the user URL box cause ADD instead of NEXT

[Jon Turney <jon.turney@dronecode.org.uk>]

On 04/12/2017 15:58, Ken Brown wrote:
> ---
>   site.cc | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/site.cc b/site.cc
> index 641a6bb..64e56a8 100644
> --- a/site.cc
> +++ b/site.cc
> @@ -759,7 +759,9 @@ bool SitePage::OnMessageCmd (int id, HWND hwndctl, UINT code)
>       {
>       case IDC_EDIT_USER_URL:
>         {
> -	// FIXME: Make Enter here cause an ADD, not a NEXT.
> +	// Set the default pushbutton to ADD if the user is entering text.
> +	if (code == EN_CHANGE)
> +	  SendMessage (GetHWND (), DM_SETDEFID, (WPARAM) IDC_BUTTON_ADD_URL, 0);
>   	break;
>         }
>       case IDC_URL_LIST:
> 

Very nice.  That fixme has been there since 2002 (and the bug probably 
longer...)

I thought perhaps we might need to reset the default control if the 
focus is moved to another control after IDC_EDIT_USER_URL so that enter 
works correctly then, but that doesn't seem to be the case.

Please apply.

The search textbox on the package chooser page needs the same fix.