<div class="gmail_quote">On Sun, Sep 19, 2010 at 2:59 PM, Dmitry Kazakov <span dir="ltr"><<a href="mailto:dimula73@gmail.com">dimula73@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br><br><div class="gmail_quote"><div><div></div><div class="h5">On Sun, Sep 19, 2010 at 4:56 PM, Sven Langkamp <span dir="ltr"><<a href="mailto:sven.langkamp@gmail.com" target="_blank">sven.langkamp@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><div></div><div><div class="gmail_quote">On Sun, Sep 19, 2010 at 2:53 PM, Dmitry Kazakov <span dir="ltr"><<a href="mailto:dimula73@gmail.com" target="_blank">dimula73@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><div></div><div><br><br><div class="gmail_quote">On Sun, Sep 19, 2010 at 4:41 PM, Sven Langkamp <span dir="ltr"><<a href="mailto:sven.langkamp@gmail.com" target="_blank">sven.langkamp@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><div class="gmail_quote">On Sun, Sep 19, 2010 at 2:28 PM, Boudewijn Rempt <span dir="ltr"><<a href="mailto:boud@valdyas.org" target="_blank">boud@valdyas.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div>On Sunday 19 September 2010, Sven Langkamp wrote:<br>
> On Sun, Sep 19, 2010 at 2:20 PM, Boudewijn Rempt <<a href="mailto:boud@valdyas.org" target="_blank">boud@valdyas.org</a>> wrote:<br>
><br>
> > On Sunday 19 September 2010, Sven Langkamp wrote:<br>
> ><br>
> > > Yes, in some places we are still missing systemLocked. The layerbox can<br>
> > also<br>
> > > be used to find them ;)<br>
> ><br>
> > :-) But I don't think systemlocked makes sense for the freehand tool,<br>
> > especially since it will cause flicker of the toolbox on eveyr stroke<br>
> ><br>
><br>
> You want it to update, but not to flicker? ;)<br>
> It can be changed by the timeout which is currently one second.<br>
><br>
<br>
</div>No, I'm not talking about the layerbox, I meant: doesn't setSystemLocked disable all the tools in the toolbox -- i.e., gray them out? If that happens on every stroke with the freehand tool the effect will be as distracting as the flickering of the paintop toolbar was after Adam's patch.<br>
</blockquote></div><br></div>Yes, they are grayed out. We could solve this by adding another parameter that would block it for very short locks like tools.<br clear="all"></blockquote></div><br></div></div>I think adding parameters is the worst thing that we can do here =(<br>
</blockquote></div><br></div></div>For what reason?<br></blockquote></div></div><div><br>Because it'll be pain to maintain. <br></div></div></blockquote><div><br>I don't think so. We the the systemLocked everywhere anyway just to be sure that is will not crash. An there wouldn't be less work to maintain it than with an updated() <br>
<br>Attached patch with the parameter change.<br></div></div>