[Cuis-dev] Morph #redrawNeeded double send

Phil B pbpublist at gmail.com
Thu Aug 22 09:13:41 PDT 2019


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...

On Thu, Aug 22, 2019 at 8:53 AM Juan Vuletich <juan at jvuletich.org> wrote:

> On 8/22/2019 4:30 AM, Phil B via Cuis-dev wrote:
> > A minor fix...
>
> Well, that was not a bug, really. #redrawNeeded should better be named
> #invalidateDisplayArea or such. What it does is to add the affected
> pixels to the list of dirty rectangles. So it is needed to be done both
> before moving it (to update the now uncovered pixels), and after moving
> it (to properly draw the morph at its new position).
>
> Thanks,
>
> --
> Juan Vuletich
> www.cuis-smalltalk.org
> https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev
> https://github.com/jvuletich
> https://www.linkedin.com/in/juan-vuletich-75611b3
> @JuanVuletich
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20190822/38097cd5/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 3848-morphPosition-comment-PhilBellalouna-2019Aug22-12h07m-pb.1.cs.st
Type: application/octet-stream
Size: 535 bytes
Desc: not available
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20190822/38097cd5/attachment.obj>


More information about the Cuis-dev mailing list