<div dir="ltr">Hi Eric,<div> as I said before, great job! Nice implementation and very well tested.</div><div> It is on github now.</div><div><br></div><div> I made a few changes:</div><div>1) Changed from cmd+P to cmd+O (as you said cmd+T is used) (Juan, we will have to talk about shortcuts I guess, we are running out of them :-) )</div><div>2) Made some formatting changes to make it more "Smalltalk oriented" :-)</div><div><br></div><div> I have a few improvements to suggest that it would be great if you could do them:</div><div>1) When the temporal to refactor is the only one, the source code keeps the temp. definition pipes, the | |. It would be great if the were removed from the source code</div><div>2) When there is another method that defines the same temporary, it would be great that the error message would say the method and class to easily find it. </div><div>What I do in this cases is to show the methods as elements in the menu so selecting them I can browse the method if necessary. Or you could show an option like "browse methods" or something like that to browse all those methods and therefore help to change those temporaries if necessary. Look for some examples in other refactorings and appliers... </div><div>3) The same thing when there are subclasses that define the inst. var.</div><div><br></div><div> Thank you again! this is a great contribution to Cuis. Keep doing more, we want more! :-)</div><div><br></div><div>Cheers!</div><div>Hernan.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 17, 2019 at 7:27 PM Eric Brandwein <<a href="mailto:brandweineric@gmail.com">brandweineric@gmail.com</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"><div dir="auto">I remember trying with Ctrl+shift+t and it being assigned to something else. I don't have the environment at hand right now, but I'll confirm later.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El vie., 17 may. 2019 14:45, Hernan Wilkinson <<a href="mailto:hernan.wilkinson@10pines.com" target="_blank">hernan.wilkinson@10pines.com</a>> escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Great! I'll take a look at it. It is really useful now that var shadowing is not allowed.</div><div>One thing, ctrl+ship+p is used in LiveTyping to show the types of the node under the cursor. What about using ctrl+shift+t ?<br></div><div><br></div><div>Hernan</div><div><br></div><div>PS:  Thank you for contributing to minimize the missing refactorings :-) </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 17, 2019 at 1:49 PM Eric Brandwein via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st" rel="noreferrer" target="_blank">cuis-dev@lists.cuis.st</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"><div dir="ltr"><div>Hi all,</div><div><br></div><div>In my view, Cuis is missing a lot of refactorings. To start with something, here's a ChangeSet that adds the TemporaryToInstanceVariable refactor, along with a ChangeSet that adds tests. I've associated the <ctrl>+<shift>+P shortcut to it.</div><div><br></div><div>To try it, just make a method with a temporary variable, place the cursor above the temporary, and press the shortcut.<br></div><div><br></div><div>Cheers,</div><div>Eric<br></div></div>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" rel="noreferrer" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail-m_5413784980187239220m_3714332335946149315gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal"><span style="font-weight:bold">Hernán Wilkinson</span><br>Agile Software Development, Teaching & Coaching</span></font></span></span></span></strong></span></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal">Phone: +54-011</span></font></span></span></span></strong></span><font face="tahoma, sans-serif" size="2">-4893-2057</font></div><div><strong style="font-family:tahoma,sans-serif;font-size:xx-small"><span style="font-size:8pt"><span style="font-size:small"><font size="2"><span style="font-weight:normal">Twitter: @HernanWilkinson</span></font></span></span></strong></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal">site: <a href="http://www.10pines.com/" style="color:rgb(17,65,112)" rel="noreferrer" target="_blank">http://www.10Pines.com</a></span></font></span></span></span></strong></span></div><div><font face="tahoma, sans-serif"><span style="border-collapse:collapse">Address: Alem 896</span></font>, Floor 6, Buenos Aires, Argentina</div></div></div></div></div></div></div></div></div></div></div></div>
</blockquote></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal"><span style="font-weight:bold">Hernán Wilkinson</span><br>Agile Software Development, Teaching & Coaching</span></font></span></span></span></strong></span></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal">Phone: +54-011</span></font></span></span></span></strong></span><font face="tahoma, sans-serif" size="2">-4893-2057</font></div><div><strong style="font-family:tahoma,sans-serif;font-size:xx-small"><span style="font-size:8pt"><span style="font-size:small"><font size="2"><span style="font-weight:normal">Twitter: @HernanWilkinson</span></font></span></span></strong></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal">site: <a href="http://www.10pines.com/" style="color:rgb(17,65,112)" target="_blank">http://www.10Pines.com</a></span></font></span></span></span></strong></span></div><div><font face="tahoma, sans-serif"><span style="border-collapse:collapse">Address: Alem 896</span></font>, Floor 6, Buenos Aires, Argentina</div></div></div></div></div></div></div></div></div></div></div></div>