<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<!--[if !mso]><style>v\:* {behavior:url(#default#VML);}
o\:* {behavior:url(#default#VML);}
w\:* {behavior:url(#default#VML);}
.shape {behavior:url(#default#VML);}
</style><![endif]--><style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
.MsoChpDefault
{mso-style-type:export-only;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style>
</head>
<body lang="EN-US" link="blue" vlink="#954F72" style="word-wrap:break-word">
<div class="WordSection1">
<p class="MsoNormal">Hi,</p>
<p class="MsoNormal">A small issue in Community (dark) theme:</p>
<p class="MsoNormal"><img width="266" height="118" style="width:2.7708in;height:1.2291in" id="Picture_x0020_5" src="cid:image001.png@01D8818F.46E48F00"></p>
<p class="MsoNormal">The class identifier (almost) not visible in this combination of colors.</p>
<p class="MsoNormal">Best,</p>
<p class="MsoNormal">Jaromir</p>
<p class="MsoNormal"><o:p> </o:p></p>
<div style="mso-element:para-border-div;border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal" style="border:none;padding:0in"><b>From: </b><a href="mailto:marcel.taeumel@hpi.de">Marcel Taeumel</a><br>
<b>Sent: </b>Thursday, June 16, 2022 14:16<br>
<b>To: </b><a href="mailto:squeak-dev@lists.squeakfoundation.org">squeak-dev</a><br>
<b>Subject: </b>Re: [squeak-dev] Review Request: fix-preference-browser-overflow.1.cs</p>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black">Hi Christoph --<o:p></o:p></span></p>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p> </o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black">> </span><span style="font-size:12.0pt;color:black">IMO the current appearance of the preference</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">> browser is not really acceptable for the release.</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p> </o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">Please elaborate. :-) I like it. Just some extra window</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">resizing needed, depending on what the user wants to</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">browse.</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p> </o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">Biggest issues are:</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">- Still not ToolBuilder-based but a nice alternative</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">- No input fields for float values</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">- Preference list needs a grid layout but we have none</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">- Input field would need minimum+preferred extent info but TableLayout does not support such layout strategies</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p> </o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">It's still a compromise. I find it acceptable for the</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">release. It looked worse in the prior releases.</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p> </o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><img border="0" width="402" height="267" style="width:4.1875in;height:2.7812in" id="f83497ea-3171-4080-bd53-31013def9a80" src="cid:image003.png@01D8818F.6C4C0F20"></span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p> </o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">Best,</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">Marcel</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
</div>
<blockquote style="border:none;border-left:solid windowtext 1.0pt;padding:0in 0in 0in 8.0pt;margin-left:0in;margin-top:15.0pt;margin-bottom:5.0pt;min-width: 500px">
<p style="margin-top:7.5pt"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:#AAAAAA">Am 16.06.2022 12:08:59 schrieb Thiede, Christoph <christoph.thiede@student.hpi.uni-potsdam.de>:<o:p></o:p></span></p>
<div>
<div id="divtagdefaultwrapper">
<p><span style="font-size:12.0pt;color:black">Hi Marcel,<o:p></o:p></span></p>
<p><span style="font-size:12.0pt;color:black"><o:p> </o:p></span></p>
<p><span style="font-size:12.0pt;color:black">this was only intended to be a quick-fix for the release, but you are right that solutions like this use to persist longer than we would love to. IMO the current appearance of the preference browser is not really
acceptable for the release. Yes, we could also only increase the #initialExtent, but this simply requires even more screen space and I considered the original preference name a little bit too verbose anyway (do we even have an official definition of "tool
windows vs windows"?).<o:p></o:p></span></p>
<p><span style="font-size:12.0pt;color:black"><o:p> </o:p></span></p>
<p><span style="font-size:12.0pt;color:black">Any ideas for a more elegant but cheap solution would be welcome.<o:p></o:p></span></p>
<p><span style="font-size:12.0pt;color:black"><o:p> </o:p></span></p>
<p><span style="font-size:12.0pt;color:black">Best,<o:p></o:p></span></p>
<p><span style="font-size:12.0pt;color:black">Christoph<o:p></o:p></span></p>
</div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><img border="0" width="648" height="2" style="width:6.75in;height:.0208in" id="Horizontal_x0020_Line_x0020_2" src="cid:image004.png@01D8818F.6C4C0F20"></span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p></o:p></span></p>
<div id="divRplyFwdMsg">
<p class="MsoNormal"><b><span style="font-size:10.0pt;color:black">Von:</span></b><span style="font-size:10.0pt;color:black"> Squeak-dev <squeak-dev-bounces@lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel<br>
<b>Gesendet:</b> Donnerstag, 16. Juni 2022 11:53:05<br>
<b>An:</b> squeak-dev<br>
<b>Betreff:</b> Re: [squeak-dev] Review Request: fix-preference-browser-overflow.1.cs</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black">
<o:p></o:p></span></p>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"> <o:p></o:p></span></p>
</div>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black">Hi Christoph --
<o:p></o:p></span></p>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p> </o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black">I don't like the reason you provide for changing the title of that preference. This kind of change is not maintainable in the long term. So, you drop the information
"tool" for some highly fragile UI optimization. This is not okay. Next time somebody increases the width of that input field for #String, your change becomes void again.<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p> </o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black">Also, you even increased the #initialExtent. So you made two very different changes to tackle the same problem. Why not just increase the #initialExtent only to support
this very configuration of preferences we have at the moment? This is not maintainable.<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p> </o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black">Your choice but this is a clear -1 from my side.<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><o:p> </o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black">Best,<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black">Marcel<o:p></o:p></span></p>
</div>
<blockquote style="border:none;border-left:solid windowtext 1.0pt;padding:0in 0in 0in 8.0pt;margin-left:0in;margin-top:15.0pt;margin-bottom:5.0pt;min-width: 500px">
<p style="margin-top:7.5pt"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:#AAAAAA">Am 16.06.2022 11:34:12 schrieb christoph.thiede@student.hpi.uni-potsdam.de <christoph.thiede@student.hpi.uni-potsdam.de>:<o:p></o:p></span></p>
</blockquote>
</div>
</div>
</blockquote>
<p class="MsoNormal" style="mso-margin-top-alt:0in;margin-right:1.0in;margin-bottom:5.0pt;margin-left:0in">
<b><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black">=============== Summary ===============</span></b><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><br>
<br>
Change Set: fix-preference-browser-overflow<br>
Date: 16 June 2022<br>
Author: Christoph Thiede<br>
<br>
Fixes default width of preference browser. The new preference 'RealEstateAgent windowScaleFactor' had such a long name that it did not fit into the standard width. Shorten the name and extend the initial window extent so that selecting the preference does no
longer offsets all items in the list.<br>
<br>
Note that this is only a quick-fix and might fail again once you change your screen size or fonts. In the long term, it would be nicer if we had something like StringMorph >> #maxWidth: for this (maybe even with an automatic ellipsis ...).<br>
<br>
<b>=============== Diff ===============</b><br>
<br>
<b>PreferenceBrowser>>initialExtent {user interface} · ct 6/16/2022 11:24 (changed)</b><br>
initialExtent<br>
"Scale here because the receiver is not built through a tool builder. See #buildPluggableWindow:."<br>
<br>
</span><s><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:blue">- ^ (660@440 * RealEstateAgent scaleFactor) rounded<br>
</span></s><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:red">+ ^ (705 @ 440 * RealEstateAgent scaleFactor) rounded</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"><br>
<br>
<b>RealEstateAgent class>>windowScaleFactor {preferences} · ct 6/16/2022 11:23 (changed)</b><br>
</span><s><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:blue">- windowScaleFactor<br>
- <preference: 'Scale Factor for Default Extent of Tool Windows'<br>
</span></s><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:red">+ windowScaleFactor
<br>
+ <preference: 'Scale Factor for Default Window Extent'<br>
</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"> categoryList: #(Morphic Tools Accessibility)<br>
</span><s><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:blue">- description: 'The extent of tool windows is tweaked for a world extent of 1024@768 (or its scaled equivalent). If you have a larger screen available, you might want
to increase a tool''s default extent. Note that you can also increase the system''s overall scale factor instead.'<br>
</span></s><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:red">+ description: 'The extent of windows is tweaked for a world extent of 1024@768 (or its scaled equivalent). If you have a larger screen available, you might want to increase
a tool''s default extent. Note that you can also increase the system''s overall scale factor instead.'<br>
</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black"> type: #String><br>
^ WindowScaleFactor ifNil: [1.0]<br>
<br>
["fix-preference-browser-overflow.1.cs"]<br>
<br>
</span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:gray">---<br>
<i>Sent from <a href="https://github.com/hpi-swa-lab/squeak-inbox-talk"><span style="color:gray">Squeak Inbox Talk</span></a></i></span><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:black">
<o:p></o:p></span></p>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</body>
</html>