Further, the current�SystemWindow&gt;&gt;addMorph:fullFrame: assumes thick borders between components, for pane splitters. �There used to be an accessor on SystemWindow, allowPaneSplitters:, that could be used to discard pane splitters and their thick borders, but this was ditched.<div>
<br></div><div>So if I want a window containing lots of morphs with no splitters and no thick borders how do I do it?</div><div><br></div><div>a) Should I use a container morph and add this to the SystemWindow?</div><div>
<br></div><div>b) should I add an override to SystemWindow that does what addMorph:fullFrame: (or addMorph:frame:) does without adding the thick borders?</div><div><br></div><div>c) should I add back the allowPaneSplitters[:] accessor and modify SystemWindow&gt;&gt;addMorph:fullFrame: to take account of allowPaneSplitters == false?</div>
<div><br></div><div>a) seems the right approach, but when I use either a Morph or a BorderedMorph as my container I don&#39;t get things laid out correctly in that morph. �Is there a standard container morph that does layout correctly? �If so, what is it?</div>
<div><br></div><div>c) is easy to do, but folks have to maintain it (it&#39;s no more than a guard around all the frae manipulation in�addMorph:fullFrame:).</div><div><br><br><div class="gmail_quote">On Wed, Jun 12, 2013 at 3:23 PM, Eliot Miranda <span dir="ltr">&lt;<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><br><div class="gmail_quote"><div><div class="h5">On Wed, Jun 12, 2013 at 2:39 PM, Eliot Miranda <span dir="ltr">&lt;<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi All,<div><br></div><div>� � in�SystemWindow&gt;&gt;addMorph:fullFrame: towards the bottom are the lines:<br><div><br></div><div><div><span style="white-space:pre-wrap">	</span>aMorph adoptPaneColor: self paneColor.</div>


<div><span style="white-space:pre-wrap">	</span>aMorph borderWidth: 1; borderColor: Color lightGray; color: Color white.</div></div><div><br></div><div>I&#39;m adding a UpdatingThreePhaseButtonMorph checkBox to my window so, via ImageMorph&gt;&gt;colors:�</div>


<div><br></div><div><div>ImageMorph&gt;&gt;color: aColor</div><div>� � � � super color: aColor.</div><div>� � � � (image depth = 1 and: [aColor isColor]) ifTrue: [</div><div>� � � � � � � � image colors: {Color transparent. aColor}.</div>


<div>� � � � � � � � self changed]</div></div><div><br></div><div>this has the effect of completely smashing the checkBox&#39;s normal image, and setting to white ScriptingSystem formAtKey: #CheckBoxOn, which is what the checkBox uses as its default image.</div>


<div><br></div><div>What business does SystemWindow have of smashing the colour of any morph added to it? �Surely this is bogus, no?</div></div></blockquote><div><br></div></div></div><div>I see the regression. �Back in 3.9 the method read</div>

<div><br></div><div><div><span style="white-space:pre-wrap">		</span>(aMorph isKindOf: ImageMorph) ifFalse:[</div><div><span style="white-space:pre-wrap">			</span>aMorph adoptPaneColor: self paneColor.</div>
<div><span style="white-space:pre-wrap">			</span>aMorph borderWidth: 2; borderColor: #inset; color: Color transparent]</div></div><div><br></div><div>Unless anyone objects I&#39;ll put back this guard. �it seems obviously correct (or rather, obviously less broken; the method also includes this priceless gem:</div>

<div><div><span style="white-space:pre-wrap">	</span>(aMorph class name = #BrowserCommentTextMorph) ifTrue:</div><div><span style="white-space:pre-wrap">		</span>[aLayoutFrame rightOffset: windowBorderWidth negated.</div>

<div><span style="white-space:pre-wrap">		</span>aLayoutFrame leftOffset: windowBorderWidth.</div><div><span style="white-space:pre-wrap">		</span>aLayoutFrame bottomOffset: windowBorderWidth negated.</div>
<div><span style="white-space:pre-wrap">		</span>aLayoutFrame topOffset: (windowBorderWidth negated) + 4].</div></div><div>).</div><span class="HOEnZb"><font color="#888888"><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div><span><font color="#888888">-- <br>best,<div>Eliot</div>
</font></span></div>
</blockquote></font></span></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><div><br></div>-- <br>best,<div>Eliot</div>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>best,<div>Eliot</div>
</div>