<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Jun 13, 2013, at 2:31 AM, Eliot Miranda <<a href="mailto:eliot.miranda@gmail.com">eliot.miranda@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Further, the current SystemWindow>>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></blockquote><div><br></div><div>We will rewrite a part of this code :).</div><div><br></div><br><blockquote type="cite"><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>>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'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'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"><<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>></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"><<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>></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>>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'm adding a UpdatingThreePhaseButtonMorph checkBox to my window so, via ImageMorph>>colors: </div>
<div><br></div><div><div>ImageMorph>>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'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'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>
</blockquote></div><br></body></html>