<div dir="auto">I've just checked it out, and though my solution is pretty ugly, I'm not entirely convinced by this one either. There's some time when the lastEditTimeStamp is nil although there has been a command. If we try to do anything with that time stamp before calling logUndoAndReplace... we would be seeing an invalid state. Instead, maybe we could add a new instance variable to act as a flag, similar to what I was doing but cleaner, without adding a new collaborator to the message. It would be similar to your solution too, just using a separate variable to signal a new command. What do you think?<div dir="auto"><br></div><div dir="auto">Cheers,</div><div dir="auto">Eric</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El vie., 6 sep. 2019 11:40, Juan Vuletich <<a href="mailto:juan@jvuletich.org">juan@jvuletich.org</a>> escribió:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 9/3/2019 11:46 PM, Eric Brandwein via Cuis-dev wrote:<br>
> Hi all,<br>
><br>
> Attached is a ChangeSet that makes it so that inserting a new line on <br>
> the SmalltalkEditor creates a new undoable command, and so undoing <br>
> only erases up until before the new line. Comments welcome!<br>
><br>
> Cheers,<br>
> Eric<br>
<br>
Thaks Eric. Now at GitHub. I did some refactoring on this, as the new <br>
parameter you added recently isn't really needed. Check the changes if <br>
curious, and ask any questions.<br>
<br>
Cheers,<br>
<br>
-- <br>
Juan Vuletich<br>
<a href="http://www.cuis-smalltalk.org" rel="noreferrer noreferrer" target="_blank">www.cuis-smalltalk.org</a><br>
<a href="https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev" rel="noreferrer noreferrer" target="_blank">https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev</a><br>
<a href="https://github.com/jvuletich" rel="noreferrer noreferrer" target="_blank">https://github.com/jvuletich</a><br>
<a href="https://www.linkedin.com/in/juan-vuletich-75611b3" rel="noreferrer noreferrer" target="_blank">https://www.linkedin.com/in/juan-vuletich-75611b3</a><br>
@JuanVuletich<br>
<br>
</blockquote></div>