<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000;text-align: left" dir="ltr">
Hi Christoph --<div><br></div><div> ><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px"> let's change it to "</span><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px">softLineWrap: true".</span></div><div><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px"><br></span></div><div><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px">No. That's even worse... Please re-read my previous response...</span></div><div><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px"><br></span></div><div><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px">> </span><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px">What is the advantage of inserting manual line breaks in all example texts?</span><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px"> </span></div><div><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px"><br></span></div><div>Small screens. Font browsing. Visually stable example texts per line.</div><div><br></div><div><img id="d453e45e-22d5-487d-8afe-f10023e5ba20" src="cid:0ef0a73e-b66c-4597-b1fd-983cf31d653a" width="277" height="223"></img><br></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 28.03.2022 19:18:20 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">
<p>Hi Marcel,</p>
<p><br>
</p>
<p>sorry, then instead of commenting it out, let's change it to "<span>softLineWrap: true".</span></p>
<p><span><br>
</span></p>
<p><span>> <span>Here, it is explicit that we do not want soft line breaks.</span></span></p>
<p><span><span><br>
</span></span></p>
<p><span><span>Why exactly not? What is the advantage of inserting manual line breaks in all example texts? I don't understand. :-)</span></span></p>
<p><span><span><br>
</span></span></p>
<p><span><span>Best,</span></span></p>
<p><span><span>Christoph</span></span></p>
</div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="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 Taeumel, Marcel<br>
<b>Gesendet:</b> Montag, 28. März 2022 18:55:28<br>
<b>An:</b> squeak-dev<br>
<b>Betreff:</b> Re: [squeak-dev] The Trunk: Collections-ct.998.mcz</span>
<div> </div>
</div>
<div>
<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000;text-align: left" dir="ltr">
Hi Christoph --
<div><br>
</div>
<div>You know that it makes sense to document the expected defaults in case those change at some point? Here, it is explicit that we do not want soft line breaks. It is similar to when somebody sets the color of a morph to its default blue. ;-) (... and here
in case somebody replaces #pluggableTextPaneSpec with #pluggableCodePaneSpec ...)</div>
<div><br>
</div>
<div>Please don't start removing init-code that happens to match the current defaults. For example, you will find "hResizing: #rigid" at several places...</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 28.03.2022 17:46:26 schrieb christoph.thiede@student.hpi.uni-potsdam.de <christoph.thiede@student.hpi.uni-potsdam.de>:</p>
<div style="font-family:Arial,Helvetica,sans-serif">Hi Marcel,<br>
<br>
sorry, I missed that. Hm ... Why do we need that #softLineWrap: in FontImporterTool>>#buildPreviewPaneWith:? If I comment out this line, all preview texts look plausible to me. But maybe I am missing something else. :-)<br>
<br>
Best,<br>
Christoph<br>
<br>
<span style="color: #808080">---<br>
</span><span style="color: #808080"><i>Sent from </i></span><span style="color: #808080"><i><a href="https://github.com/hpi-swa-lab/squeak-inbox-talk"><u><span style="color: #808080">Squeak Inbox Talk</span></u></a></i></span><br>
<br>
On 2022-03-28T16:38:50+02:00, marcel.taeumel@hpi.de wrote:<br>
<br>
> Hi Christoph --<br>
> <br>
> > Removes obsolete line breaks from the comment.<br>
> <br>
> Please re-add those line breaks ASAP. This is no dialog text but exemplary filler text. This must be layouted for now. See Font Importer -> Source Code as an example.<br>
> <br>
> Thanks. :-)<br>
> <br>
> <br>
> Best,<br>
> Marcel<br>
> Am 28.03.2022 13:57:29 schrieb commits at source.squeak.org <commits at source.squeak.org>:<br>
> Christoph Thiede uploaded a new version of Collections to project The Trunk:<br>
> http://source.squeak.org/trunk/Collections-ct.998.mcz<br>
> <br>
> ==================== Summary ====================<br>
> <br>
> Name: Collections-ct.998<br>
> Author: ct<br>
> Time: 28 March 2022, 1:56:52.242267 pm<br>
> UUID: 74080f41-16ed-dc43-8f7c-1cd16bc91db7<br>
> Ancestors: Collections-ct.997<br>
> <br>
> Adds literal arrays and scaled decimals to the smalltalk postcard. Removes obsolete line breaks from the comment.<br>
> <br>
> =============== Diff against Collections-ct.997 ===============<br>
> <br>
> Item was changed:<br>
> ----- Method: Text class>>codeSample (in category 'filler text') -----<br>
> codeSample<br>
> <br>
> ^ 'exampleWithNumber: x<br>
> + "A method that illustrates every part of Smalltalk method syntax including primitives. It has unary, binary, and keyboard messages; declares arguments and temporaries; accesses a global variable (but not an instance variable); uses literals (array, nested
array, character, symbol, string, integer, float, scaled decimal, and byte array); uses the pseudo variables nil, true, false, self, super, and thisContext; shows that within a literal array nil, true, and false are symbols not pseudo variables; and has sequence,
assignment, return, cascade, and tuple (array) creation. It has both zero argument and one argument blocks, and has a block temporary."<br>
> - "A method that illustrates every part of Smalltalk method syntax<br>
> - including primitives. It has unary, binary, and keyboard messages,<br>
> - declares arguments and temporaries, accesses a global variable<br>
> - (but not an instance variable), uses literals (array, character, symbol,<br>
> - string, integer, float, byte array), uses the pseudo variables nil, true,<br>
> - false, self, super, and thisContext, shows that within a literal array<br>
> - nil, true and false are symbols not pseudo variables, and has sequence,<br>
> - assignment, return, cascade, and tuple (array) creation. It has both<br>
> - zero argument and one argument blocks, and has a block temporary."<br>
> <br>
> | y |<br>
> true & false not & (nil isNil) ifFalse: [self halt].<br>
> y := self size + super size.<br>
> + #($a #a ''a'' "a" (1 1.0 1.0s2) nil true false), { #[65]. thisContext. nil. true. false }<br>
> - #($a #a ''a'' "a" 1 1.0 nil true false), { #[65]. thisContext. nil. true. false }<br>
> do: [ :each | | class |<br>
> class := each class.<br>
> Transcript<br>
> show: (class name);<br>
> show: '' ''].<br>
> ^ x<br>
> <br>
> <br>
> -------------- next part --------------<br>
> An HTML attachment was scrubbed...<br>
> URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220328/27278fe8/attachment.html><br>
> <br>
> </div>
</blockquote>
</div>
</div>
</div></blockquote></div>