[squeak-dev] Review Request: fix-preference-browser-overflow.1.cs

Thiede, Christoph Christoph.Thiede at student.hpi.uni-potsdam.de
Thu Jun 16 17:08:45 UTC 2022


Thanks for the feedback! I think this color check is just a minimum, we cannot print Shout-styled strings reliably on an unknown color.


Fixed via System-ct.1362, PreferenceBrowser-ct.152, and PreferenceBrowser-ct.153.


Best,

Christoph

________________________________
Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel
Gesendet: Donnerstag, 16. Juni 2022 16:42:48
An: squeak-dev
Betreff: Re: [squeak-dev] Review Request: fix-preference-browser-overflow.1.cs

We have no actual "color support" for "source code on list-selection color" in all the UI themes. Please keep the workarounds to a minimum here. Thanks.

Best,
Marcel

Am 16.06.2022 16:24:09 schrieb Jaromir Matas <mail at jaromir.net>:
Definitely improved :)
[cid:image005.png at 01D8819C.C4968630]

--

Jaromír Matas

mail at jaromir.net

From: Thiede, Christoph<mailto:Christoph.Thiede at student.hpi.uni-potsdam.de>
Sent: Thursday, June 16, 2022 15:47
To: The general-purpose Squeak developers list<mailto:squeak-dev at lists.squeakfoundation.org>
Subject: Re: [squeak-dev] Review Request: fix-preference-browser-overflow.1.cs


Hi Jaromir,

> A small issue in Community (dark) theme:
> [cid:dd7447d9-20ee-4e8a-8b18-da4b44484b06]
> The class identifier (almost) not visible in this combination of colors.


please see PreferenceBrowser-ct.151 (inbox). :-)



Best,

Christoph

Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Jaromir Matas <mail at jaromir.net>
Gesendet: Donnerstag, 16. Juni 2022 14:43:25
An: The general-purpose Squeak developers list
Betreff: Re: [squeak-dev] Review Request: fix-preference-browser-overflow.1.cs

Hi,
A small issue in Community (dark) theme:
[cid:image001.png at 01D8818F.46E48F00]
The class identifier (almost) not visible in this combination of colors.
Best,
Jaromir

From: Marcel Taeumel<mailto:marcel.taeumel at hpi.de>
Sent: Thursday, June 16, 2022 14:16
To: squeak-dev<mailto:squeak-dev at lists.squeakfoundation.org>
Subject: Re: [squeak-dev] Review Request: fix-preference-browser-overflow.1.cs

Hi Christoph --

> IMO the current appearance of the preference
> browser is not really acceptable for the release.

Please elaborate. :-) I like it. Just some extra window
resizing needed, depending on what the user wants to
browse.

Biggest issues are:
- Still not ToolBuilder-based but a nice alternative
- No input fields for float values
- Preference list needs a grid layout but we have none
- Input field would need minimum+preferred extent info but TableLayout does not support such layout strategies

It's still a compromise. I find it acceptable for the
release. It looked worse in the prior releases.

[cid:image003.png at 01D8818F.6C4C0F20]

Best,
Marcel

Am 16.06.2022 12:08:59 schrieb Thiede, Christoph <christoph.thiede at student.hpi.uni-potsdam.de>:

Hi Marcel,



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"?).



Any ideas for a more elegant but cheap solution would be welcome.



Best,

Christoph

Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel
Gesendet: Donnerstag, 16. Juni 2022 11:53:05
An: squeak-dev
Betreff: Re: [squeak-dev] Review Request: fix-preference-browser-overflow.1.cs

Hi Christoph --

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.

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.

Your choice but this is a clear -1 from my side.

Best,
Marcel

Am 16.06.2022 11:34:12 schrieb christoph.thiede at student.hpi.uni-potsdam.de <christoph.thiede at student.hpi.uni-potsdam.de>:
=============== Summary ===============

Change Set:        fix-preference-browser-overflow
Date:            16 June 2022
Author:            Christoph Thiede

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.

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 ...).

=============== Diff ===============

PreferenceBrowser>>initialExtent {user interface} · ct 6/16/2022 11:24 (changed)
initialExtent
    "Scale here because the receiver is not built through a tool builder. See #buildPluggableWindow:."

-     ^ (660 at 440 * RealEstateAgent scaleFactor) rounded
+     ^ (705 @ 440 * RealEstateAgent scaleFactor) rounded

RealEstateAgent class>>windowScaleFactor {preferences} · ct 6/16/2022 11:23 (changed)
- windowScaleFactor
-     <preference: 'Scale Factor for Default Extent of Tool Windows'
+ windowScaleFactor
+     <preference: 'Scale Factor for Default Window Extent'
        categoryList: #(Morphic Tools Accessibility)
-         description: 'The extent of tool windows is tweaked for a world extent of 1024 at 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.'
+         description: 'The extent of windows is tweaked for a world extent of 1024 at 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.'
        type: #String>
    ^ WindowScaleFactor ifNil: [1.0]

["fix-preference-browser-overflow.1.cs"]

---
Sent from Squeak Inbox Talk<https://github.com/hpi-swa-lab/squeak-inbox-talk>


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220616/651ae52a/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 97ABF51FBF3E40088E3F5E8D6400E6AC.png
Type: image/png
Size: 11094 bytes
Desc: 97ABF51FBF3E40088E3F5E8D6400E6AC.png
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220616/651ae52a/attachment-0006.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 3D8B02D8731D4418995AFBBB9AAFC2C0.png
Type: image/png
Size: 44102 bytes
Desc: 3D8B02D8731D4418995AFBBB9AAFC2C0.png
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220616/651ae52a/attachment-0007.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 362D62FFFE6441E4A8404CFDAF65C999.png
Type: image/png
Size: 144 bytes
Desc: 362D62FFFE6441E4A8404CFDAF65C999.png
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220616/651ae52a/attachment-0008.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pastedImage.png
Type: image/png
Size: 35918 bytes
Desc: pastedImage.png
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220616/651ae52a/attachment-0009.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 99D8B0145CC84A699E4FC4B9E6892D84.png
Type: image/png
Size: 7526 bytes
Desc: 99D8B0145CC84A699E4FC4B9E6892D84.png
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220616/651ae52a/attachment-0010.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 9AC709DC5EF54A2FBB76A70A719E1106.png
Type: image/png
Size: 144 bytes
Desc: 9AC709DC5EF54A2FBB76A70A719E1106.png
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220616/651ae52a/attachment-0011.png>


More information about the Squeak-dev mailing list