[squeak-dev] The Inbox: Compiler-cmm.480.mcz

Marcel Taeumel marcel.taeumel at hpi.de
Tue Jul 19 12:20:49 UTC 2022


> 75em may also be a good limit, now that Number even understands such a selector.

Nope. We need something independent from a scale factor and thus the default font size as pretty-printed format can be checked in. Better use "66 characters", not 75em. No rendering/font properties. For multi-line comments, use the line-break algorithm as in Text >> #withNoLineLongerThan:.

Please read the commentary in
TextStyle >> #compositionWidthForNumChars

Best,
Marcel
Am 14.07.2022 19:55:04 schrieb tim Rowledge <tim at rowledge.org>:


> On 2022-07-14, at 10:48 AM, Jakob Reschke wrote:
>
> 75em may also be a good limit, now that Number even understands such a selector.
> https://baymard.com/blog/line-length-readability

That's an excellent point; extra-long lines are really annoying to read in general.

>
> But seeing Tim's examples: before we start tweaking the indentation of comments by the pretty printer, which would have it insert tabs or spaces or anything else, better implement that proposal to display comment lines left-aligned with the opening ", without requiring indentation characters in the source text. Remember that other thread about whether or not to have line breaks in comments? Somebody posted a screenshot from Dolphin Smalltalk that shows what I am referring to.
>
> To leave the comments untouched by the pretty-printer seems reasonable to me for now.

Oh, I'm not wanting to have any tabs or spaces inserted - I want the formatting to be live and use the left indent. Shout does all that work to colour (etc) the text so why not use the fact that it detects comments.


tim
--
tim Rowledge; tim at rowledge.org; http://www.rowledge.org/tim
Useful random insult:- His seat back is not in the full upright and locked position.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220719/8052762d/attachment.html>


More information about the Squeak-dev mailing list