<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">That’s interesting! I didn’t know Cmd+Shift+e. so I think we should directly perform a search using the selected text instead of opening a new FillingTheBlank morph in that case (I think that is what you suggested).<div class=""><br class=""></div><div class="">In that case we should change the behavior of cmd+shift+e too, shouldn’t we?.</div><div class=""><br class=""></div><div class="">I didn’t understand how to reproduce those bugs you mention.</div><div class=""><br class=""></div><div class="">Thanks!</div><div class=""><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On 10 Jul 2019, at 11:20, Hernan Wilkinson <<a href="mailto:hernan.wilkinson@10pines.com" class="">hernan.wilkinson@10pines.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Hi Gaston,<div class=""> I like the idea of using cmd+shit+f to find but:</div><div class="">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 class="">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 class="">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 class="">4) If you press cancel when asking what to look for, it looks anyway when it should not.</div><div class=""><br class=""></div><div class="">So, my suggestion is:</div><div class="">1) do not remove cmd+T, it is not necessary at this time</div><div class="">2) make cmd+F do a lookup in the method source not in strings defined in methods, cmd+E does that already</div><div class="">3) add the postcript to initialize the editor (you can do it from the changes window)</div><div class="">4) add (F) to the "Methos source with it" to the smalltalk editor menu </div><div class=""><br class=""></div><div class="">Is anybody against changing the cmd+F short cut?</div><div class=""><br class=""></div><div class="">Hernan.</div></div><br class=""><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" class="">cuis-dev@lists.cuis.st</a>> wrote:<br class=""></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" class="">Hi all! I've realized that we have two funny shortcuts (Cmd+Shift+f / Cmd+Shift+t) to insert #ifTrue: and #ifFalse:<br class="">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 class=""><div class=""><br class=""></div><div class="">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 class=""><br class=""></div><div class="">The attached chage set includes a postcript to update Editor so it would update the shortcuts when you file in it.</div><div class=""><br class=""></div><div class="">Let me know what do you think, have a good night!</div><div class=""><br class=""></div><div class="">Cheers,</div><div class="">Gastón Caruso</div></div>
-- <br class="">
Cuis-dev mailing list<br class="">
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank" class="">Cuis-dev@lists.cuis.st</a><br class="">
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank" class="">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br class="">
</blockquote></div><br clear="all" class=""><div class=""><br class=""></div>-- <br class=""><div dir="ltr" class="gmail_signature"><div dir="ltr" class=""><div class=""><div dir="ltr" class=""><div dir="ltr" class=""><div dir="ltr" class=""><div dir="ltr" class=""><div dir="ltr" class=""><div dir="ltr" class=""><div dir="ltr" class=""><div dir="ltr" class=""><div class=""><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse" class=""><strong class=""><span style="font-size:8pt" class=""><span class=""><span style="font-size:small" class=""><font size="2" class=""><span style="font-weight:normal" class=""><span style="font-weight:bold" class="">Hernán Wilkinson</span><br class="">Agile Software Development, Teaching & Coaching</span></font></span></span></span></strong></span></div><div class=""><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse" class=""><strong class=""><span style="font-size:8pt" class=""><span class=""><span style="font-size:small" class=""><font size="2" class=""><span style="font-weight:normal" class="">Phone: +54-011</span></font></span></span></span></strong></span><font face="tahoma, sans-serif" size="2" class="">-4893-2057</font></div><div class=""><strong style="font-family:tahoma,sans-serif;font-size:xx-small" class=""><span style="font-size:8pt" class=""><span style="font-size:small" class=""><font size="2" class=""><span style="font-weight:normal" class="">Twitter: @HernanWilkinson</span></font></span></span></strong></div><div class=""><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse" class=""><strong class=""><span style="font-size:8pt" class=""><span class=""><span style="font-size:small" class=""><font size="2" class=""><span style="font-weight:normal" class="">site: <a href="http://www.10pines.com/" style="color:rgb(17,65,112)" target="_blank" class="">http://www.10Pines.com</a></span></font></span></span></span></strong></span></div><div class=""><font face="tahoma, sans-serif" class=""><span style="border-collapse:collapse" class="">Address: Alem 896</span></font>, Floor 6, Buenos Aires, Argentina</div></div></div></div></div></div></div></div></div></div></div></div>
</div></blockquote></div><br class=""></div></body></html>