This is the mail archive of the cygwin-apps mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH setup] (Usability improvement) Implement half-second wait for user to finish typing before searching packages


On Aug  1 23:17, Ronald Ramos wrote:
> commit 357c1e7576586349efb8514dc9d8d03950e225ee
> Author: Ronald Ramos <ronjoe223@gmail.com>
> Date:   Mon Aug 1 23:05:44 2016 -0400
> 
>     * proppage.h (PropertyPage)
>         New member OnTimerMessage (delegates similarly to OnMouseWheel)
> 
>     * proppage.cc
>         (DialogProc) Added handling of WM_TIMER
> 
>     * choose.h (ChooserPage)
>         (OnTimerMessage) New function prototype
>         (timer_id) New member variable
>         Added DEFINE-ed default values for timer_id and search timer delay
>         Reorganized private members for consistency
> 
>     * choose.cc
>         (constructor) Initialize timer_id
>         (OnMessageCmd) Replaced search-refresh with a SetTimer
>         (OnSearchTimer) New; contains search-refresh removed from
> OnMessageCmd

Neat!  Applied.  Personally I would rather see a shorter timeout like,
say, 300ms, but I guess we shouldn't take for granted that everybody is
typing fast.  Even with 500ms timeout it's now much more convenient than
the old "search after each keypress" method.

However.  Your MUA seems to scramble patches as part of the mail text,
e.g., it adds line breaks which break `patch' or `git am'.

It would be nice if you could fix this.  Alternatively, just attach the
output from `git format-patch' as mail attachment.  That usually works
nicely.


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

Attachment: signature.asc
Description: PGP signature


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]