<div dir="ltr"><div>Hi Christoph,</div><div>I do not agree.</div><div></div><div>Most certainly, if some fraction is un-initialized, it's because we only specified the offset and want an absolute layout.</div><div>Likewise, if offset are un-initialized, it's because we only specified a purely proportional layout and just want that.
<div>This makes zero the best default value (least surprising).</div></div><div><br></div><div>if someone wants a full layout default, then he just has to specify LayoutFrame full, or something like that.<br></div><div><br></div><div>Look in inbox, this goes very well.</div><div>Let's eradicate those inaesthetic ifNil: [], YAGNI.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le lun. 17 févr. 2020 à 16:55, Thiede, Christoph <<a href="mailto:Christoph.Thiede@student.hpi.uni-potsdam.de">Christoph.Thiede@student.hpi.uni-potsdam.de</a>> a écrit :<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 dir="ltr">
<div id="gmail-m_-8859176588262384398divtagdefaultwrapper" style="font-size:12pt;color:rgb(0,0,0);font-family:Calibri,Helvetica,sans-serif" dir="ltr">
<div id="gmail-m_-8859176588262384398divtagdefaultwrapper" 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">
<p>Hm ... sorry, this does not yet convince me. :)</p>
<p><br>
</p>
<p>I think the best idea of a default LayoutFrame is fullscreen. This would be LayoutFrame fractions: (0 @ 0 rect: 1 @ 1) offsets: (0 @ 0 rect: 0 @ 0).</p>
<p><br>
</p>
<p>Apart from that, there is a big number of nil checks in LayoutFrame. If we introduce something like #fixup, we should also get rid of all these nil checks. Or at least, define a consistent fallback for a nil value, so either we keep the nil values as in <span>Graphics-ct.426,
 or we introduce #fixUp as in <span>Graphics-nice.425 AND change or delete #minExtentFrom: & Co.</span></span></p>
<p><span><span><br>
</span></span></p>
<p><span><span>Best,</span></span></p>
<p><span><span>Christoph</span></span></p>
<p><span><span><br>
</span></span></p>
<div id="gmail-m_-8859176588262384398Signature">
<div id="gmail-m_-8859176588262384398divtagdefaultwrapper" 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">
<div><font size="2" color="#808080"></font></div>
</div>
</div>
</div>
</div>
<hr style="display:inline-block;width:98%">
<div id="gmail-m_-8859176588262384398divRplyFwdMsg" 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 Nicolas Cellier <<a href="mailto:nicolas.cellier.aka.nice@gmail.com" target="_blank">nicolas.cellier.aka.nice@gmail.com</a>><br>
<b>Gesendet:</b> Montag, 17. Februar 2020 16:06:38<br>
<b>An:</b> The general-purpose Squeak developers list<br>
<b>Betreff:</b> Re: [squeak-dev] The Inbox: Graphics-jr.425.mcz</font>
<div> </div>
</div>
<div>
<div dir="ltr">But these are unsent messages in trunk...<br>
</div>
<br>
<div class="gmail_quote">
<div dir="ltr" class="gmail_attr">Le lun. 17 févr. 2020 à 15:52, Christoph Thiede <<a href="mailto:christoph.thiede@student.hpi.uni-potsdam.de" target="_blank">christoph.thiede@student.hpi.uni-potsdam.de</a>> a écrit :<br>
</div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> Do we rely anywhere on having nil fractions?<br>
<br>
Yes, apparently we do, have a look at #minWidthFrom:<br>
<br>
...<br>
right := rightFraction ifNil: [1.0].<br>
...<br>
<br>
Just fixed this in Graphics-ct.426.<br>
<br>
<br>
<br>
--<br>
Sent from: <a href="http://forum.world.st/Squeak-Dev-f45488.html" rel="noreferrer" target="_blank">
http://forum.world.st/Squeak-Dev-f45488.html</a><br>
<br>
</blockquote>
</div>
</div>
</div>
</div>

<br>
</blockquote></div>