<div dir="ltr">Hi Gaston,<div> I like the idea of using cmd+shit+f to find but:</div><div>1) The short cut cmd+shift+e is already assigned to find string within methods. It works in the context of an Smalltalk Editor and it does the same as you did</div><div>2) Your change introduced two bugs when pressing cmd+T or cmd+F in the browser's editor because you removed #displayIfFalse: and #displayIfTrue: but did not add a postscript to initialize the short cut table (Editor initialize)</div><div>3) I think that instead of looking for string containing what you entered, it should look for source code containing what you entered</div><div>4) If you press cancel when asking what to look for, it looks anyway when it should not.</div><div><br></div><div>So, my suggestion is:</div><div>1) do not remove cmd+T, it is not necessary at this time</div><div>2) make cmd+F do a lookup in the method source not in strings defined in methods, cmd+E does that already</div><div>3) add the postcript to initialize the editor (you can do it from the changes window)</div><div>4) add (F) to the "Methos source with it" to the smalltalk editor menu </div><div><br></div><div>Is anybody against changing the cmd+F short cut?</div><div><br></div><div>Hernan.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jul 9, 2019 at 9:42 PM Gastón Caruso via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st">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">Hi all! I've realized that we have two funny shortcuts (Cmd+Shift+f / Cmd+Shift+t) to insert #ifTrue: and #ifFalse:<br>I don't know if someone is using it or if it is there just for historical reasons, but I've removed them and replaced the last one (cmd+shift+f) with the "Method Source with it" option, which I use a lot.<br><div><br></div><div>I didn't find the way to avoid duplicated code on PasteUpMorph and TextEditor (see #openSourceFinder), also I don't know how to classify the new messages.</div><div><br></div><div>The attached chage set includes a postcript to update Editor so it would update the shortcuts when you file in it.</div><div><br></div><div>Let me know what do you think, have a good night!</div><div><br></div><div>Cheers,</div><div>Gastón Caruso</div></div>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="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_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>