<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000">
Hi Chirstoph,<div><br></div><div>not sure I like the wording #debugFromContextThat:title:. On the plus side, we can remove the explicit reference to #Process.</div><div><br></div><div>Maybe consider what is already in the system:</div><div><br></div><div>Morph >> #firstOwnerSuchThat:</div><div>Object >> #hasLiteralSuchThat:</div><div>ChangeList >> #selectSuchThat:</div><div><br></div><div>Maybe #debugContextSuchThat:(withTitle:)?</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 01.10.2020 22:47:49 schrieb Thiede, Christoph <christoph.thiede@student.hpi.uni-potsdam.de>:</p><div style="font-family:Arial,Helvetica,sans-serif">
<div id="divtagdefaultwrapper" style="font-size: 12pt;color: #000000;font-family: Calibri,Helvetica,sans-serif" dir="ltr">
<div id="divtagdefaultwrapper" style="font-size: 12pt;color: rgb(0, 0, 0);font-family: Calibri, Helvetica, sans-serif, EmojiFont, "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols" dir="ltr">
<p>Hi Marcel,</p>
<p><br>
</p>
<p>thank you for reviewing all this stuff! :-)</p>
<p><br>
</p>
<p>Might this be a good point to push the discussion from <a href="http://forum.world.st/The-Inbox-System-ct-1149-mcz-td5114102.html" class="OWAAutoLink" id="LPlnk887838" previewremoved="true">http://forum.world.st/The-Inbox-System-ct-1149-mcz-td5114102.html</a> again?
It would allow us to rewrite the #debugAction implementation here as follows:</p>
<p><br>
</p>
<p></p>
</div>
<blockquote style="margin: 0 0 0 40px;border: none;padding: 0px;min-width: 500px">
<div style="font-size: 12pt;color: rgb(0, 0, 0);font-family: Calibri, Helvetica, sans-serif, EmojiFont, "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols" dir="ltr">
<div>debugAction</div>
</div>
<div style="font-size: 12pt;color: rgb(0, 0, 0);font-family: Calibri, Helvetica, sans-serif, EmojiFont, "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols" dir="ltr">
<div><br>
</div>
</div>
<div style="font-size: 12pt;color: rgb(0, 0, 0);font-family: Calibri, Helvetica, sans-serif, EmojiFont, "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols" dir="ltr">
<div> action ifNil: [^ super debugAction].</div>
</div>
<div style="font-size: 12pt;color: rgb(0, 0, 0);font-family: Calibri, Helvetica, sans-serif, EmojiFont, "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols" dir="ltr">
<div> [self performAction]</div>
</div>
<div style="font-size: 12pt;color: rgb(0, 0, 0);font-family: Calibri, Helvetica, sans-serif, EmojiFont, "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols" dir="ltr">
<div> debugFromContextThat: [<span style="font-size: 12pt">context closure = action]</span></div>
</div>
<div style="font-size: 12pt;color: rgb(0, 0, 0);font-family: Calibri, Helvetica, sans-serif, EmojiFont, "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols" dir="ltr">
<div><span style="font-size: 12pt"> title: </span><span style="font-size: 12pt">('Debug button action "{1}" in model "{2}"' format: {self label. self target printString}).</span></div>
</div>
</blockquote>
<div style="font-size: 12pt;color: rgb(0, 0, 0);font-family: Calibri, Helvetica, sans-serif, EmojiFont, "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols" dir="ltr">
<div><span style="font-size: 12pt"><br>
</span></div>
<div><span style="font-size: 12pt">Best,</span></div>
<div><span style="font-size: 12pt">Christoph</span></div>
<p></p>
<div id="Signature">
<div id="divtagdefaultwrapper" dir="ltr" style="font-size: 12pt;color: rgb(0, 0, 0);font-family: Calibri, Helvetica, sans-serif, EmojiFont, "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols">
<div name="divtagdefaultwrapper" style="font-family: Calibri,Arial,Helvetica,sans-serif;font-size: ;margin: 0">
<div>
<div class="_rp_T4" id="Item.MessagePartBody">
<div class="_rp_U4 ms-font-weight-regular ms-font-color-neutralDark rpHighlightAllClass rpHighlightBodyClass" id="Item.MessageUniqueBody" style="font-family:wf_segoe-ui_normal,"Segoe UI","Segoe WP",Tahoma,Arial,sans-serif,serif,EmojiFont">
<div dir="ltr">
<div id="divtagdefaultwrapper"><span style="font-family: Calibri,Helvetica,sans-serif,EmojiFont,Apple Color Emoji,Segoe UI Emoji,NotoColorEmoji,Segoe UI Symbol,Android Emoji,EmojiSymbols">
<div id="Signature">
<div style="margin:0px"><span style="">
<div><span style="font-size: 12pt;color: black"><span style="font-size: 12pt"><a href="http://www.hpi.de/" target="_blank" rel="noopener noreferrer" id="LPNoLP"><font size="2"><span id="LPlnk909538"><font color="#757B80"></font></span></font></a></span></span></div>
</span></div>
</div>
</span></div>
</div>
</div>
</div>
</div>
<div><span style="font-size: 10pt;color: #808080"></span></div>
</div>
</div>
</div>
<br>
<br>
<div style="color: rgb(0, 0, 0);">
<div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><span style="font-family: Calibri, sans-serif;color: #000000"><b>Von:</b> Squeak-dev <squeak-dev-bounces@lists.squeakfoundation.org> im Auftrag von commits@source.squeak.org <commits@source.squeak.org><br>
<b>Gesendet:</b> Donnerstag, 1. Oktober 2020 16:06 Uhr<br>
<b>An:</b> squeak-dev@lists.squeakfoundation.org; packages@lists.squeakfoundation.org<br>
<b>Betreff:</b> [squeak-dev] The Trunk: ToolBuilder-Morphic-ct.252.mcz</span>
<div> </div>
</div>
</div>
<span style="font-size: 10pt"><span style="font-size: 10pt">
<div class="PlainText">Marcel Taeumel uploaded a new version of ToolBuilder-Morphic to project The Trunk:<br>
<a href="http://source.squeak.org/trunk/ToolBuilder-Morphic-ct.252.mcz">http://source.squeak.org/trunk/ToolBuilder-Morphic-ct.252.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: ToolBuilder-Morphic-ct.252<br>
Author: ct<br>
Time: 20 January 2020, 8:57:46.404972 pm<br>
UUID: 2b311bcc-31ca-dc4d-8ebf-7a1cad07624e<br>
Ancestors: ToolBuilder-Morphic-mt.251<br>
<br>
Implement "browse/debug button action" properly on PluggabeButtonMorphPlus<br>
<br>
=============== Diff against ToolBuilder-Morphic-mt.251 ===============<br>
<br>
Item was added:<br>
+ ----- Method: PluggableButtonMorphPlus>>browseImplementationOfActionSelector (in category 'debug menu') -----<br>
+ browseImplementationOfActionSelector<br>
+ <br>
+ action ifNotNil: [<br>
+ ^ action outerContext method browse].<br>
+ ^ super browseImplementationOfActionSelector!<br>
<br>
Item was added:<br>
+ ----- Method: PluggableButtonMorphPlus>>debugAction (in category 'debug menu') -----<br>
+ debugAction<br>
+ <br>
+ action ifNil: [^ super debugAction].<br>
+ (Process<br>
+ forBlock: [self performAction]<br>
+ runUntil: [:context | context closure = action])<br>
+ debugWithTitle: ('Debug button action "{1}" in model "{2}"' format: {self label. self target printString}).!<br>
<br>
<br>
</div>
</span></span></div>
</div>
</div>
</div></blockquote></div>