<div dir="ltr"><div>Thanks for maintaining compatibility with TruffleSqueak, Christoph! :)</div><div><br></div><div>-- Fabio<br></div><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 26, 2022 at 3:57 PM Thiede, Christoph <<a href="mailto:Christoph.Thiede@student.hpi.uni-potsdam.de">Christoph.Thiede@student.hpi.uni-potsdam.de</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>

<div id="gmail-m_959069532729140667divtagdefaultwrapper" style="font-size:12pt;color:rgb(0,0,0);font-family:Calibri,Helvetica,sans-serif" dir="ltr">
<p>Hi Marcel,</p>
<p><br>
</p>
<p>thank you! In doubt, I would always use #allSubInstancesDo: instead of #allInstancesDo: for the same reason we (almost) always use #isKindOf: rather than #isMemberOf: - because any subclass might just be an implementation detail of the original protocol.
 I think this is an amount of speculative generality that does not hurt. :-) There are indeed some subclasses of Workspace (e.g., I think TruffleSqueak does this), and by default, their styling should just work like in any normal workspace.</p>
<p><br>
</p>
<p>Best,</p>
<p>Christoph</p>
</div>
<hr style="display:inline-block;width:98%">
<div id="gmail-m_959069532729140667divRplyFwdMsg" dir="ltr"><font style="font-size:11pt" face="Calibri, sans-serif" color="#000000"><b>Von:</b> Squeak-dev <<a href="mailto:squeak-dev-bounces@lists.squeakfoundation.org" target="_blank">squeak-dev-bounces@lists.squeakfoundation.org</a>> im Auftrag von Taeumel, Marcel<br>
<b>Gesendet:</b> Dienstag, 26. April 2022 13:52:15<br>
<b>An:</b> squeak-dev<br>
<b>Betreff:</b> Re: [squeak-dev] The Inbox: Tools-ct.1147.mcz</font>
<div> </div>
</div>
<div>
<div id="gmail-m_959069532729140667__MailbirdStyleContent" style="font-size:10pt;font-family:Arial;color:rgb(0,0,0);text-align:left" dir="ltr">
Hi Christoph --
<div></div>
<div><br>
</div>
<div>There are no subclasses of Workspace. I am not sure whether we even want to encourage to have some. At least not in this way. Hmm... #allInstancesDo: should be sufficient for this purpose.</div>
<div><br>
</div>
<div>Anyway: +1 :)</div>
<div><br>
</div>
<div>Best,</div>
<div>Marcel</div>
<div><br>
</div>
<div><br>
</div>
<blockquote 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:rgb(170,170,170);margin-top:10px">Am 26.04.2022 12:12:48 schrieb <a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a> <<a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a>>:</p>
<div style="font-family:Arial,Helvetica,sans-serif">A new version of Tools was added to project The Inbox:<br>
<a href="http://source.squeak.org/inbox/Tools-ct.1147.mcz" target="_blank">http://source.squeak.org/inbox/Tools-ct.1147.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: Tools-ct.1147<br>
Author: ct<br>
Time: 26 April 2022, 12:12:36.942702 pm<br>
UUID: 11944f29-f575-7b41-824d-71e945afa0d2<br>
Ancestors: Tools-tpr.1146<br>
<br>
Updates workspace styles after toggling #shouldStyle preference. This also improves the liveness in the preference wizard.<br>
<br>
=============== Diff against Tools-tpr.1146 ===============<br>
<br>
Item was changed:<br>
----- Method: Workspace class>>shouldStyle: (in category 'preferences') -----<br>
shouldStyle: aBoolean<br>
<br>
+ ShouldStyle := aBoolean.<br>
+ <br>
+ self allSubInstancesDo: [:workspace | workspace changed: #style].!<br>
- ShouldStyle := aBoolean!<br>
<br>
<br>
</div>
</blockquote>
</div>
</div>
</div>

<br>
</blockquote></div></div>