<div dir="ltr">Doh... of course you're right.  A bit more testing would have 'shown' that ;-)  Short of renaming the method, how about this comment to make it clear...</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 22, 2019 at 8:53 AM Juan Vuletich <<a href="mailto:juan@jvuletich.org">juan@jvuletich.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 8/22/2019 4:30 AM, Phil B via Cuis-dev wrote:<br>
> A minor fix...<br>
<br>
Well, that was not a bug, really. #redrawNeeded should better be named <br>
#invalidateDisplayArea or such. What it does is to add the affected <br>
pixels to the list of dirty rectangles. So it is needed to be done both <br>
before moving it (to update the now uncovered pixels), and after moving <br>
it (to properly draw the morph at its new position).<br>
<br>
Thanks,<br>
<br>
-- <br>
Juan Vuletich<br>
<a href="http://www.cuis-smalltalk.org" rel="noreferrer" target="_blank">www.cuis-smalltalk.org</a><br>
<a href="https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev" rel="noreferrer" target="_blank">https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev</a><br>
<a href="https://github.com/jvuletich" rel="noreferrer" target="_blank">https://github.com/jvuletich</a><br>
<a href="https://www.linkedin.com/in/juan-vuletich-75611b3" rel="noreferrer" target="_blank">https://www.linkedin.com/in/juan-vuletich-75611b3</a><br>
@JuanVuletich<br>
<br>
</blockquote></div>