On Wed, Apr 29, 2009 at 7:58 AM, Steve Borho <[email protected]> wrote:
> On Wed, Apr 29, 2009 at 12:22 AM,  <[email protected]> wrote:
>> Please double check the "ui refresh" code in _msg_config.  This seems
>> to be the pattern used elsewhere, but it makes me nervous.  Also,
>> please note that I've got the config set at the repo, not global, level.
>
> Pushed.
>
> The ui refresh code is fine for now, I'll probably revisit that
> project wide after
> 1.3 is released and the new ui class is settled.
>
> As for configuring local or repo, that is also ok for now.  I would like to 
> fix
> that issue in a more general way as well.
>
> Two small issues that need to be addressed:
>
> 1) In thgconfig you need to document the default behavior of those two 
> settings.
> ie: they default to 0, which implies no enforcement.

I'll fix that up today.

> 2) the word wrap algorithm is adding whitespace to the end of each line.

This was actually deliberate.  When someone backspaces from the
beginning of the next line, the single added space will still separate
the words.  Happy to axe it if you feel that is best.


I'll also do the commit checks today.

------------------------------------------------------------------------------
Register Now & Save for Velocity, the Web Performance & Operations 
Conference from O'Reilly Media. Velocity features a full day of 
expert-led, hands-on workshops and two days of sessions from industry 
leaders in dedicated Performance & Operations tracks. Use code vel09scf 
and Save an extra 15% before 5/3. http://p.sf.net/sfu/velocityconf
_______________________________________________
Tortoisehg-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/tortoisehg-develop

Reply via email to