[squeak-dev] ChangeSorter preamble eating seems to break broken

tim Rowledge tim at rowledge.org
Thu Dec 30 20:49:49 UTC 2021



> On 2021-12-30, at 1:05 AM, Marcel Taeumel <marcel.taeumel at hpi.de> wrote:
> 
> Hi Tim --
> 
> > Marcel, what was your intent for that change?
> 
> It was gone by accident. I wanted to minimize the interaction/clicks when working/managing change sets.

Ah, right. I can see why you might want that.

> 
> Could we spawn an extra window just like with "postscripts"? (Yellow-click on change set, edit/add potsscript, new window appears)

Well oddly enough, the (unused) ChangeSorter>>#editPreamble method purports to do just that. In ChangeSorter>>#unshiftedChangeSetMenu: it looks as if at one point the expectation was to send it but #addPreamble got used instead. No idea why.

However, it looks like it it really easy to solve this. The problem is that ChangeSorter>>#addPreamble has been using the massive hammer of `self showChangeSet: myChangeSet` since 1999 when al that is required is
`		self changed: #messageList.
		self setContents.
		self contentsChanged`

Try this - 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: TPR-ChangeBrowser-preamble.1.cs
Type: application/octet-stream
Size: 743 bytes
Desc: not available
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20211230/e4cac07b/attachment.obj>
-------------- next part --------------



tim
--
tim Rowledge; tim at rowledge.org; http://www.rowledge.org/tim
Strange OpCodes: STOP: No Op




More information about the Squeak-dev mailing list