What if we change SectorMorph&gt;&gt;#angle: to work like this?<div><br></div><div><div>angle: aNumber </div><div><span class="Apple-tab-span" style="white-space:pre">        </span>angle = aNumber</div><div><span class="Apple-tab-span" style="white-space:pre">                </span>ifTrue: [^ self].</div>

<div><span class="Apple-tab-span" style="white-space:pre">        </span>angle := aNumber \\ 360.</div><div><span class="Apple-tab-span" style="white-space:pre">        </span>(angle = 0 and: [aNumber ~= 0])</div><div><span class="Apple-tab-span" style="white-space:pre">                </span>ifTrue: [angle := 360].</div>

<div><span class="Apple-tab-span" style="white-space:pre">        </span>self update</div><div><br></div><div>I think this would allow showing a full circle when the angle is 360 degrees and it won&#39;t get out of sync.</div><div>

<br></div><div>Cheers,</div><div>Richo</div><br><div class="gmail_quote">On Sat, Dec 8, 2012 at 5:01 PM, David Corking <span dir="ltr">&lt;<a href="mailto:lists@dcorking.com" target="_blank">lists@dcorking.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I take it you see the angle get out of sync: after each revolution, it<br>
is one degree too small.<br>
<br>
I posted a quick fix to the inbox, MorphicExtras-dcorking.75. I made<br>
SectorMorph#angle revert to zero when it reaches 360. On reflection, I<br>
think my change will do more harm than good.<br>
<br>
If a learner makes an automated pie chart, it is perfectly reasonable<br>
for a sector to grow to 100% or 360 degrees. The user will expect the<br>
sector to become a full circle, and my &#39;fix&#39; breaks that. Therefore I<br>
think Richo&#39;s original code: modulo 361 - is reasonable.<br>
<br>
Perhaps we can provide a more subtle fix? Can we use this hack?<br>
   angle := aNumber \\  360.000000000001.<br>
It still gets out of sync, but much more slowly.<br>
<br>
David<br>
_______________________________________________<br>
etoys-dev mailing list<br>
<a href="mailto:etoys-dev@squeakland.org">etoys-dev@squeakland.org</a><br>
<a href="http://lists.squeakland.org/mailman/listinfo/etoys-dev" target="_blank">http://lists.squeakland.org/mailman/listinfo/etoys-dev</a><br>
</blockquote></div><br></div>