Hi Marcel, thanks for the fix!


Specific to the FontChooserTool: Why do we need

self changed: #pointSizeList.

in the method you patched (#selectedPointSizeIndex:)?


> No need to bring back those hacks in PluggableListMorph again.

So the problem is that in past models were encouraged not to signal a selection change after a list change was signaled?
Then, in theory, it would be possible to do something like this:

PluggableListMorph >> update: aSymbol

aSymbol == getListSelector ifTrue: [
self updateList.
model isDec2019Compatible ifFalse: [
self update: getIndexSelector]].
aSymbol == getIndexSelector ifTrue: [^ self updateListSelection].
...

I would bet that out there are many more models that won't work after the 5.3 release. I'm afraid there is also a world outside of Trunk :(
It just would be sad if five years later, someone wants to try out any cool Squeak package developed for Squeak 5.1 that uses lists, and, without special knowledge of the bug, he or she will fail to use it for this trivial reason.
On the other hand, of course, I see the added complexity of such a versioning mechanism ...

Best,
Christoph


________________________________
Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel
Gesendet: Montag, 6. Januar 2020 09:53 Uhr
An: John Pfersich via Squeak-dev; Chris Muller
Betreff: Re: [squeak-dev] 5.3 font size selection bug

> Although it appears to register the click, the list selection does not update.

Fixed in Morphic-mt.1614.

> It would be nice to keep downward compatible.

In general, yes. In this case, challenging because that view -> model communication can be performance critical. Models (or tools) that forgot to send some changed-event, can easily be fixed. No need to bring back those hacks in PluggableListMorph again.

Best,
Marcel

Am 04.01.2020 02:23:33 schrieb Thiede, Christoph <christoph.thiede at student.hpi.uni-potsdam.de>:

Nice catch, can reproduce it. I saw this effect at different places during the last weeks (e. g., changesets, but this is fixed now) - did we (Marcel) change the relevant list protocol? It would be nice to keep downward compatible.


Christoph

________________________________
Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Chris Muller <asqueaker at gmail.com>
Gesendet: Freitag, 3. Januar 2020 23:31:52
An: squeak dev
Betreff: [squeak-dev] 5.3 font size selection bug

Appearance... | system fonts... | button font... |

then click on one of the point sizes in the righthand pane.  Although it appears to register the click, the list selection does not update.  Arrows too, no effect...

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20200106/97dc23a4/attachment.html>