<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000">
Hi Chris,<div><br></div><div>it still sets the search string. No change in interaction here. This guards against semantic changes in #findAgain in the future.</div><div><br></div><div>#findAgain might overwrite the search string. #findAgainNow does not. So, in combination with #setSearch:, better use #findAgainNow instead of #findAgain.</div><div><br></div><div>Best,</div><div>Marcel</div><div class="mb_sig"></div>
<blockquote class="history_container" type="cite" style="border-left-style: solid;border-width: 1px;margin-top: 20px;margin-left: 0px;padding-left: 10px;min-width: 500px">
<p style="color: #AAAAAA; margin-top: 10px;">Am 24.07.2019 04:31:48 schrieb Chris Muller <asqueaker@gmail.com>:</p><div style="font-family:Arial,Helvetica,sans-serif">
<meta http-equiv="Content-Type" content="text/html; charset=utf-8"><div dir="ltr">Hi Marcel,<div><br></div><div>To clarify "more robust", is this: when I select a method in a MessageSet of "senders of at:" and it highlights "at:" for me, _don't_ make that the search string for Cmd-g?</div><div><br></div><div>Or is this something else? 'Cuz I often want to bounce to the _next_ sender in the same method via Cmd-g...</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 22, 2019 at 5:07 AM <<a href="mailto:commits@source.squeak.org">commits@source.squeak.org</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;min-width: 500px">Marcel Taeumel uploaded a new version of Morphic to project The Trunk:<br>
<a href="http://source.squeak.org/trunk/Morphic-mt.1495.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/trunk/Morphic-mt.1495.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: Morphic-mt.1495<br>
Author: mt<br>
Time: 22 July 2019, 12:07:10.880271 pm<br>
UUID: 24e42526-45ab-4631-9cd6-521e7f95ea2c<br>
Ancestors: Morphic-mt.1494<br>
<br>
Makes auto-select in pluggable text morphs more robust.<br>
<br>
=============== Diff against Morphic-mt.1494 ===============<br>
<br>
Item was changed:<br>
----- Method: PluggableTextMorph>>update: (in category 'updating') -----<br>
update: aSymbol <br>
aSymbol ifNil: [^self].<br>
aSymbol == #flash ifTrue: [^self flash].<br>
<br>
aSymbol == getTextSelector<br>
ifTrue: [<br>
self setText: self getText.<br>
getSelectionSelector<br>
ifNotNil: [self setSelection: self getSelection].<br>
^ self].<br>
aSymbol == getSelectionSelector <br>
ifTrue: [^self setSelection: self getSelection].<br>
<br>
aSymbol == #acceptChanges ifTrue: [^ self accept].<br>
aSymbol == #revertChanges ifTrue: [^ self cancel].<br>
<br>
(aSymbol == #autoSelect and: [getSelectionSelector notNil]) <br>
ifTrue: <br>
[self handleEdit: <br>
[(textMorph editor)<br>
abandonChangeText; "no replacement!!"<br>
setSearch: model autoSelectString;<br>
+ findAgainNow "do not reset search string"]].<br>
- findAgain]].<br>
aSymbol == #clearUserEdits ifTrue: [^self hasUnacceptedEdits: false].<br>
aSymbol == #wantToChange <br>
ifTrue: <br>
[self canDiscardEdits ifFalse: [^self promptForCancel].<br>
^self].<br>
aSymbol == #appendEntry <br>
ifTrue: <br>
[self handleEdit: [self appendEntry].<br>
^self refreshWorld].<br>
aSymbol == #appendEntryLater<br>
ifTrue: [self handleEdit: [self appendEntry]].<br>
aSymbol == #clearText <br>
ifTrue: <br>
[self handleEdit: [self changeText: Text new].<br>
^self refreshWorld].<br>
aSymbol == #bs <br>
ifTrue: <br>
[self handleEdit: [self bsText].<br>
^self refreshWorld].<br>
aSymbol == #codeChangedElsewhere <br>
ifTrue: <br>
[self hasEditingConflicts: true.<br>
^self changed].<br>
aSymbol == #saveContents<br>
ifTrue:<br>
[^self saveContentsInFile].<br>
aSymbol == #showContents<br>
ifTrue:<br>
[^ self scrollToTop].<br>
!<br>
<br>
<br>
</blockquote></div>
</div></blockquote></div>