<div dir="ltr">Hi all,<div dir="auto">I don't recommend mixing cosmetic changes with functional changes, unless cosmetic changes are really scarce. My experience is that it is a recipe for switching the attention focus out of the functional changes, and as such it is parasiting at best or obstructing at worse the review process.</div><div dir="auto">Also, we should not deny the value of cosmetic changes. They have some value. They require attention and review too, even if they won't break production.<br></div><div><br></div><div dir="auto">We might want to amend tools. I hate loosing history, and I value the contextual analysis of certain design decision. But the frequency of such incursion into the past is gradually fading away. MC requires history for merging, but merging a too distant branch is becoming close to impossible anyway for human review. I tried it with Pharo some years ago, after repairing the conscious erasing of history that happened at each Pharo release (which is a very aggressive way to solve the problem, the worse-is-better-way)..</div><div>So having the whole history in memory is questionable. It's not the whole history, what about Smalltalk 72, 76, 80, the change-set and update stream era... We have to set a limit.<br></div><div><br></div><div>For browsing the history, I don't know what is worse, having a long list of small changes, or shorter list of squashed (in git parlance) unrelated changes. But this should be handled in the interface IMO. If we prefer grouping, this should be done by the browsing tool. For my own usage, it would have helped if the tool could filter the list by date range, modified class, method, pattern matching on commit message, ...</div><div><br></div><div><div>Somehow related, it happens that I group several changes in one commit, I did it recently in VMMaker, one change for solving Newspeak generated code, the other for avoiding generating variable named bool, which were obstructing my attempt to compile legacy Mac OS  carbon VM. I did regret, because the first change introduced a slip in generated code that made the build fail in the branch dedicated to second change... Using CI automation is another reason why we should value separation of concerns. If I want to merge the compile_legacy_MacOS branch, one may ask in the future why the hell the totally unrelated Newspeak change was introduced in this branch. That's an anti-pattern, especially if we use explicitely named feature branches (which we avoid currently in MC, partly for lack of tool support).</div><div><br></div></div><div>Last, MC does not behave so badly. I have far worse user experience using sourcetree on windows for file based projects on a gitlab server where load balancing configuration is not optimal, I can tell you that we should stop bashing ourselves ;)<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le ven. 15 nov. 2019 à 04:39, David T. Lewis <<a href="mailto:lewis@mail.msen.com">lewis@mail.msen.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Changing the subject line.<br>
<br>
Small commits do generate more noise, but I find that it is<br>
easier to read, assess, and manage the changes if the commits<br>
are small. I have spent a good deal of time doing this for<br>
<a href="http://www.squeaksource.com/TrunkUpdateStreamV3" rel="noreferrer" target="_blank">http://www.squeaksource.com/TrunkUpdateStreamV3</a> and my perspective<br>
is based largely on that experience.<br>
<br>
If I were to point to something in need of improvement, I would say<br>
that is the performance of our Monticello tools.<br>
<br>
I *love* having the MC change management directly in the image where<br>
everything is directly accessible, browseable, and minimally dependent<br>
on external tools. But I have to admit that the tools are slow.<br>
Processing the update stream is slow, comparing versions is slow,<br>
and reading packages is slow. And I strongly suspect that if all of<br>
these things were instantaniously fast, then none of us would care<br>
about how big or small the commits were. <br>
<br>
I may be wrong, but I have to suspect that some profiling of common<br>
use cases, such as updating Squeak from the server, might identify<br>
some huge opportunities for improvement.<br>
<br>
But even if those improvements never happen, I think that small<br>
commits are a good thing.<br>
<br>
Dave<br>
<br>
<br>
On Thu, Nov 14, 2019 at 09:32:51AM +0100, Marcel Taeumel wrote:<br>
> > ...??piddly little fixes ...<br>
> > ... such small changes are not worth burdening everyone's life ...<br>
> > ...????tiny improvements ...<br>
> <br>
> Since I merged those commits and had to read and asses them, I can say that the granularity of them was fine. Except for that one duplication for the help in Freecell.<br>
> <br>
> Let's focus on "recategorize methods". Yes, one could easily do that when fixing another issue. Unless it is a bigger quest such as Patrick's (pre) recent efforts to clean up lots of categories across classes.<br>
> <br>
> Best,<br>
> Marcel<br>
> Am 13.11.2019 20:20:41 schrieb Chris Muller <<a href="mailto:asqueaker@gmail.com" target="_blank">asqueaker@gmail.com</a>>:<br>
> Guys, would you please consider "batching up" piddly little fixes like "recatgorized one method" this into fewer, chunkier commits with other fixes??? Such small changes are not worth burdening everyone's life with an additional 2MB of memory, disk, network and CPU overhead -- in *every single image* they use going forward, forever.?? ??Please let that sink in for a moment.<br>
> <br>
> Even ignoring that, it also bloats the readability of the history.?? There's no use case where timestamping and versioning tiny improvements like this is useful, only micro harmful.<br>
> <br>
> Best,<br>
> ?? Chris<br>
> <br>
> On Wed, Nov 13, 2019 at 6:33 AM Marcel Taeumel <<a href="mailto:marcel.taeumel@hpi.de" target="_blank">marcel.taeumel@hpi.de</a> [mailto:<a href="mailto:marcel.taeumel@hpi.de" target="_blank">marcel.taeumel@hpi.de</a>]> wrote:<br>
> <br>
> Hi Christoph,<br>
> <br>
> I know. We should come up with a solution for that. Maybe re-print the commit messages of all ancestors as soon as the number of ancestors is > 1.<br>
> <br>
> Best,<br>
> Marcel<br>
> Am 13.11.2019 12:51:06 schrieb Thiede, Christoph <<a href="mailto:christoph.thiede@student.hpi.uni-potsdam.de" target="_blank">christoph.thiede@student.hpi.uni-potsdam.de</a> [mailto:<a href="mailto:christoph.thiede@student.hpi.uni-potsdam.de" target="_blank">christoph.thiede@student.hpi.uni-potsdam.de</a>]>:<br>
> Just for interest, is it usual & desired behavior that when installing these updates, the update??log does not include??any of my commit messages?<br>
> Von: Squeak-dev <<a href="mailto:squeak-dev-bounces@lists.squeakfoundation.org" target="_blank">squeak-dev-bounces@lists.squeakfoundation.org</a> [mailto:<a href="mailto:squeak-dev-bounces@lists.squeakfoundation.org" target="_blank">squeak-dev-bounces@lists.squeakfoundation.org</a>]> im Auftrag von <a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a> [mailto:<a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a>] <<a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a> [mailto:<a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a>]><br>
> Gesendet: Mittwoch, 13. November 2019 12:15:14<br>
> An: <a href="mailto:squeak-dev@lists.squeakfoundation.org" target="_blank">squeak-dev@lists.squeakfoundation.org</a> [mailto:<a href="mailto:squeak-dev@lists.squeakfoundation.org" target="_blank">squeak-dev@lists.squeakfoundation.org</a>]; <a href="mailto:packages@lists.squeakfoundation.org" target="_blank">packages@lists.squeakfoundation.org</a> [mailto:<a href="mailto:packages@lists.squeakfoundation.org" target="_blank">packages@lists.squeakfoundation.org</a>]<br>
> Betreff: [squeak-dev] The Trunk: EToys-mt.368.mcz<br>
> ??<br>
> Marcel Taeumel uploaded a new version of EToys to project The Trunk:<br>
> <a href="http://source.squeak.org/trunk/EToys-mt.368.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/trunk/EToys-mt.368.mcz</a> [<a href="http://source.squeak.org/trunk/EToys-mt.368.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/trunk/EToys-mt.368.mcz</a>]<br>
> <br>
> ==================== Summary ====================<br>
> <br>
> Name: EToys-mt.368<br>
> Author: mt<br>
> Time: 13 November 2019, 12:15:07.673043 pm<br>
> UUID: 3f6873fa-5111-ac4a-a108-9031e19e5f40<br>
> Ancestors: EToys-kfr.363, EToys-ct.354, EToys-ct.355, EToys-ct.356, EToys-ct.357, EToys-ct.358, EToys-ct.359, EToys-ct.360, EToys-ct.364, EToys-ct.365, EToys-ct.367<br>
> <br>
> Merge! Merge! Merge! Various fixes in Etoys-related places.<br>
> <br>
> =============== Diff against EToys-kfr.363 ===============<br>
> <br>
> Item was added:<br>
> + ----- Method: Form>>scaledToWidth: (in category '*Etoys-Squeakland-scaling, rotation') -----<br>
> + scaledToWidth: newWidth<br>
> +?????????????? "Answer the receiver, scaled such that it has the desired width."<br>
> +<br>
> +?????????????? newWidth = self width ifTrue: [^ self].<br>
> +?????????????? ^self magnify: self boundingBox by: (newWidth / self width) smoothing: 2.<br>
> + !<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: FreeCell>>help (in category 'actions') -----<br>
> ?? help<br>
> +<br>
> +?????????????? self helpText editWithLabel: 'FreeCell Help'.!<br>
> -?????????????? | window helpMorph |<br>
> -?????????????? window := SystemWindow labelled: 'FreeCell Help' translated.<br>
> -?????????????? window model: self.<br>
> -?????????????? helpMorph := (PluggableTextMorph new editString: self helpText) lock.<br>
> -?????????????? window<br>
> -?????????????????????????????? addMorph: helpMorph<br>
> -?????????????????????????????? frame: (0 @ 0 extent: 1 @ 1).<br>
> -?????????????? window openInWorld!<br>
> <br>
> Item was added:<br>
> + ----- Method: MovingEyeMorph>>color: (in category 'accessing') -----<br>
> + color: aColor<br>
> +<br>
> +?????????????? super color: aColor.<br>
> +??????????????<br>
> +?????????????? "Migrate old instances."<br>
> +?????????????? inner color: Color transparent.<br>
> +??????????????<br>
> +?????????????? "Keep iris visible."<br>
> +?????????????? aColor = iris color<br>
> +?????????????????????????????? ifTrue: [iris borderWidth: 1; borderColor: aColor negated]<br>
> +?????????????????????????????? ifFalse: [iris borderWidth: 0].!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: MovingEyeMorph>>initialize (in category 'initialization') -----<br>
> ?? initialize<br>
> ???????????????? "initialize the state of the receiver"<br>
> ???????????????? super initialize.<br>
> ???????????????? ""<br>
> ???????????????? inner := EllipseMorph new.<br>
> +?????????????? inner color: Color transparent.<br>
> -?????????????? inner color: self color.<br>
> ???????????????? inner extent: (self extent * (1.0 @ 1.0 - IrisSize)) asIntegerPoint.<br>
> -?????????????? inner borderColor: self color.<br>
> ???????????????? inner borderWidth: 0.<br>
> ?? ""<br>
> ???????????????? iris := EllipseMorph new.<br>
> ???????????????? iris color: Color white.<br>
> ???????????????? iris extent: (self extent * IrisSize) asIntegerPoint.<br>
> ?? ""<br>
> ???????????????? self addMorphCentered: inner.<br>
> ???????????????? inner addMorphCentered: iris.<br>
> ?? ""<br>
> ???????????????? self extent: 26 @ 33!<br>
> <br>
> Item was added:<br>
> + ----- Method: MovingEyeMorph>>irisColor (in category 'accessing') -----<br>
> + irisColor<br>
> +<br>
> +?????????????? ^ iris color!<br>
> <br>
> Item was added:<br>
> + ----- Method: MovingEyeMorph>>irisColor: (in category 'accessing') -----<br>
> + irisColor: aColor<br>
> +<br>
> +?????????????? iris color: aColor.<br>
> +??????????????<br>
> +?????????????? "Keep iris visible."<br>
> +?????????????? aColor = self color<br>
> +?????????????????????????????? ifTrue: [iris borderWidth: 1; borderColor: aColor negated]<br>
> +?????????????????????????????? ifFalse: [iris borderWidth: 0].!<br>
> <br>
> Item was added:<br>
> + ----- Method: MovingEyeMorph>>irisPos (in category 'accessing') -----<br>
> + irisPos<br>
> +<br>
> +?????????????? ^ iris position!<br>
> <br>
> Item was changed:<br>
> + ----- Method: MovingEyeMorph>>irisPos: (in category 'accessing') -----<br>
> - ----- Method: MovingEyeMorph>>irisPos: (in category 'as yet unclassified') -----<br>
> ?? irisPos: cp<br>
> ??<br>
> ???????????????? | a b theta x y |<br>
> ???????????????? theta := (cp - self center) theta.<br>
> ???????????????? a := inner width // 2.<br>
> ???????????????? b := inner height // 2.<br>
> ???????????????? x := a * (theta cos).<br>
> ???????????????? y := b * (theta sin).<br>
> ???????????????? iris position: ((x@y) asIntegerPoint) + self center - (iris extent // 2).!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: MovingEyeMorph>>step (in category 'stepping and presenter') -----<br>
> ?? step<br>
> ???????????????? | cp |<br>
> ???????????????? cp := self globalPointToLocal: self world primaryHand position.<br>
> ???????????????? (inner containsPoint: cp)<br>
> ???????????????????????????????? ifTrue: [iris position: (cp - (iris extent // 2))]<br>
> +?????????????????????????????? ifFalse: [self irisPos: cp].!<br>
> -?????????????????????????????? ifFalse: [self irisPos: cp].<br>
> -?????????????? self changed "cover up gribblies if embedded in Flash"!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: NewVariableDialogMorph>>decimalPlaces (in category 'accessing') -----<br>
> ?? decimalPlaces<br>
> ???????????????? ^ decimalPlacesButton<br>
> ???????????????????????????????? ifNil: [Utilities<br>
> ???????????????????????????????????????????????????????????????? decimalPlacesForFloatPrecision: (self targetPlayer<br>
> +?????????????????????????????????????????????????????????????????????????????? defaultFloatPrecisionFor: self varAcceptableName asSetterSelector)]<br>
> -?????????????????????????????????????????????????????????????????????????????? defaultFloatPrecisionFor: (Utilities getterSelectorFor: self varAcceptableName))]<br>
> ???????????????????????????????? ifNotNil: [:button| button label asNumber]!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: Player>>newScriptorAround: (in category 'viewer') -----<br>
> ?? newScriptorAround: aPhrase<br>
> ???????????????? "Sprout a scriptor around aPhrase, thus making a new script.?? aPhrase may either be a PhraseTileMorph (classic tiles 1997-2001) or a SyntaxMorph (2001 onward)"<br>
> ??<br>
> ???????????????? | aScriptEditor aUniclassScript tw blk |<br>
> ?? Cursor wait showWhile: [<br>
> ???????????????? aUniclassScript := self class permanentUserScriptFor: self unusedScriptName player: self.<br>
> ???????????????? aScriptEditor := aUniclassScript instantiatedScriptEditorForPlayer: self.<br>
> ??<br>
> ???????????????? Preferences universalTiles ifTrue: [<br>
> ???????????????????????????????? aScriptEditor install.<br>
> ???????????????????????????????? "aScriptEditor hResizing: #shrinkWrap;<br>
> ???????????????????????????????????????????????? vResizing: #shrinkWrap;<br>
> ???????????????????????????????????????????????? cellPositioning: #topLeft;<br>
> ???????????????????????????????????????????????? setProperty: #autoFitContents toValue: true."<br>
> ???????????????????????????????? aScriptEditor insertUniversalTiles.?? "Gets an empty SyntaxMorph for a MethodNode"<br>
> +?????????????????????????????? tw := aScriptEditor findA: ScrollPane.<br>
> -?????????????????????????????? tw := aScriptEditor findA: TwoWayScrollPane.<br>
> ???????????????????????????????? aPhrase ifNotNil:<br>
> ???????????????????????????????????????????????? [blk := (tw scroller findA: SyntaxMorph "MethodNode") findA: BlockNode.<br>
> ???????????????????????????????????????????????? blk addMorphFront: aPhrase.<br>
> ???????????????????????????????????????????????? aPhrase accept.<br>
> ???????????????????????????????? ].<br>
> ???????????????????????????????? SyntaxMorph setSize: nil andMakeResizable: aScriptEditor.<br>
> ???????????????? ] ifFalse: [<br>
> ???????????????????????????????? aPhrase<br>
> ???????????????????????????????????????????????????????????????? ifNotNil: [aScriptEditor phrase: aPhrase]???????????? "does an install"<br>
> ???????????????????????????????????????????????????????????????? ifNil: [aScriptEditor install]<br>
> ???????????????? ].<br>
> ???????????????? self class allSubInstancesDo: [:anInst | anInst scriptInstantiationForSelector: aUniclassScript selector].<br>
> ???????????????????????????????? "The above assures the presence of a ScriptInstantiation for the new selector in all siblings"<br>
> ???????????????? self updateScriptsCategoryOfViewers.<br>
> ?? ].<br>
> ???????????????? ^ aScriptEditor!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: Preferences class>>initializePreferencePanel:in: (in category '*Etoys-Squeakland-preferences panel') -----<br>
> ?? initializePreferencePanel: aPanel in: aPasteUpMorph<br>
> ???????????????? "Initialize the given Preferences panel. in the given pasteup, which is the top-level panel installed in the container window.?? Also used to reset it after some change requires reformulation"<br>
> ??<br>
> ???????????????? | tabbedPalette controlPage aColor aFont maxEntriesPerCategory tabsMorph anExtent?? prefObjects |<br>
> ???????????????? aPasteUpMorph removeAllMorphs.<br>
> ??<br>
> ???????????????? aFont := Preferences standardListFont.<br>
> +?????????????? aColor := aPanel windowColorToUse.<br>
> -?????????????? aColor := aPanel defaultBackgroundColor.<br>
> ???????????????? tabbedPalette := TabbedPalette newSticky.<br>
> ???????????????? tabbedPalette dropEnabled: false.<br>
> ???????????????? (tabsMorph := tabbedPalette tabsMorph) color: aColor darker;<br>
> ?????????????????????????????????? highlightColor: Color red regularColor: Color brown darker darker.<br>
> ???????????????? tabbedPalette on: #mouseDown send: #yourself to: #().<br>
> ???????????????? maxEntriesPerCategory := 0.<br>
> ???????????????? self listOfCategories do:<br>
> ???????????????????????????????? [:aCat |<br>
> ???????????????????????????????????????????????? controlPage := AlignmentMorph newColumn beSticky color: aColor.<br>
> ???????????????????????????????????????????????? controlPage on: #mouseDown send: #yourself to: #().<br>
> ???????????????????????????????????????????????? controlPage dropEnabled: false.<br>
> ???????????????????????????????????????????????? controlPage borderColor: aColor;<br>
> ?????????????????????????????????????????????????????????????????? layoutInset: 4.<br>
> ???????????????????????????????????????????????? (prefObjects := self preferenceObjectsInCategory: aCat) do:<br>
> ???????????????????????????????????????????????????????????????? [:aPreference | | button |<br>
> ???????????????????????????????????????????????????????????????????????????????? button := aPreference representativeButtonWithColor: Color white inPanel: aPanel.<br>
> ???????????????????????????????????????????????????????????????????????????????? button ifNotNil: [controlPage addMorphBack: button]].<br>
> ???????????????????????????????????????????????? controlPage setNameTo: aCat asString.<br>
> ???????????????????????????????????????????????? aCat = #?<br>
> ???????????????????????????????????????????????????????????????? ifTrue: [aPanel addHelpItemsTo: controlPage].<br>
> ???????????????????????????????????????????????? tabbedPalette addTabFor: controlPage font: aFont.<br>
> ???????????????????????????????????????????????? aCat = 'search results' ifTrue:<br>
> ???????????????????????????????????????????????????????????????? [(tabbedPalette tabNamed: aCat) setBalloonText:<br>
> ???????????????????????????????????????????????????????????????????????????????? 'Use the ? category to find preferences by keyword; the results of your search will show up here' translated].<br>
> ???????????????????????????????? maxEntriesPerCategory := maxEntriesPerCategory max: prefObjects size].<br>
> ???????????????? tabbedPalette selectTabNamed: '?'.<br>
> ???????????????? tabsMorph rowsNoWiderThan: aPasteUpMorph width.<br>
> ???????????????? aPasteUpMorph on: #mouseDown send: #yourself to: #().<br>
> ???????????????? anExtent := aPasteUpMorph width @ (490 max: (25 + tabsMorph height + (24 * maxEntriesPerCategory))).<br>
> ???????????????? aPasteUpMorph extent: anExtent.<br>
> ???????????????? aPasteUpMorph color: aColor.<br>
> ???????????????? aPasteUpMorph???????? addMorphBack: tabbedPalette.!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: ScriptEditorMorph>>autoFitOnOff (in category 'menu') -----<br>
> ?? autoFitOnOff<br>
> ???????????????? "Toggle between auto fit to size of code and manual resize with scrolling"<br>
> ???????????????? | tw |<br>
> +?????????????? (tw := self findA: ScrollPane) ifNil: [^ self].<br>
> -?????????????? (tw := self findA: TwoWayScrollPane) ifNil: [^ self].<br>
> ???????????????? (self hasProperty: #autoFitContents)<br>
> ???????????????????????????????? ifTrue: [self removeProperty: #autoFitContents.<br>
> ???????????????????????????????????????????????? self hResizing: #rigid; vResizing: #rigid]<br>
> ???????????????????????????????? ifFalse: [self setProperty: #autoFitContents toValue: true.<br>
> ???????????????????????????????????????????????? self hResizing: #shrinkWrap; vResizing: #shrinkWrap].<br>
> ???????????????? tw layoutChanged!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: ScriptEditorMorph>>extent: (in category 'other') -----<br>
> ?? extent: x<br>
> ??<br>
> ???????????????? | newExtent tw menu |<br>
> ???????????????? newExtent := x max: self minWidth @ self minHeight.<br>
> +?????????????? (tw := self findA: ScrollPane) ifNil:<br>
> -?????????????? (tw := self findA: TwoWayScrollPane) ifNil:<br>
> ???????????????????????????????? ["This was the old behavior"<br>
> ???????????????????????????????? ^ super extent: newExtent].<br>
> ??<br>
> ???????????????? (self hasProperty: #autoFitContents) ifTrue: [<br>
> ???????????????????????????????? menu := MenuMorph new defaultTarget: self.<br>
> ???????????????????????????????? menu addUpdating: #autoFitString target: self action: #autoFitOnOff.<br>
> ???????????????????????????????? menu addTitle: 'To resize the script, uncheck the box below' translated.<br>
> ???????????????????????????????? menu popUpEvent: nil in: self world???????? .<br>
> ???????????????????????????????? ^ self].<br>
> ??<br>
> ???????????????? "Allow the user to resize to any size"<br>
> ???????????????? tw extent: ((newExtent x max: self firstSubmorph width)<br>
> ???????????????????????????????????????????????????????????????? @ (newExtent y - self firstSubmorph height)) - (self borderWidth * 2) + (-4 @ -4).?? "inset?"<br>
> ???????????????? ^ super extent: newExtent!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: ScriptEditorMorph>>hibernate (in category 'other') -----<br>
> ?? hibernate<br>
> ???????????????? "Possibly delete the tiles, but only if using universal tiles."<br>
> ??<br>
> ???????????????? | tw |<br>
> ???????????????? Preferences universalTiles ifFalse: [^self].<br>
> +?????????????? (tw := self findA: ScrollPane) isNil<br>
> -?????????????? (tw := self findA: TwoWayScrollPane) isNil<br>
> ???????????????????????????????? ifFalse:<br>
> ???????????????????????????????????????????????? [self setProperty: #sizeAtHibernate toValue: self extent.???????????? "+ tw xScrollerHeight"<br>
> ???????????????????????????????????????????????? submorphs size > 1 ifTrue: [tw delete]]!<br>
> <br>
> Item was added:<br>
> + ----- Method: SpectrumAnalyzerMorph>>fftSize: (in category 'accessing') -----<br>
> + fftSize: aSize<br>
> +<br>
> +?????????????? | on |<br>
> +?????????????? on := soundInput isRecording.<br>
> +?????????????? self stop.<br>
> +?????????????? fft := FFT new: aSize.<br>
> +?????????????? self resetDisplay.<br>
> +?????????????? on ifTrue: [self start].!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: SpectrumAnalyzerMorph>>setFFTSize (in category 'menu and buttons') -----<br>
> ?? setFFTSize<br>
> ???????????????? "Set the size of the FFT used for frequency analysis."<br>
> ??<br>
> +?????????????? | aMenu sz |<br>
> -?????????????? | aMenu sz on |<br>
> ???????????????? aMenu := CustomMenu new title: ('FFT size (currently {1})' translated format:{fft n}).<br>
> ???????????????? ((7 to: 10) collect: [:n | 2 raisedTo: n]) do:[:r | aMenu add: r printString action: r].<br>
> ???????????????? sz := aMenu startUp.<br>
> ???????????????? sz ifNil: [^ self].<br>
> +?????????????? self fftSize: sz.!<br>
> -?????????????? on := soundInput isRecording.<br>
> -?????????????? self stop.<br>
> -?????????????? fft := FFT new: sz.<br>
> -?????????????? self resetDisplay.<br>
> -?????????????? on ifTrue: [self start].<br>
> - !<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: SyntaxMorph class>>setSize:andMakeResizable: (in category 'as yet unclassified') -----<br>
> ?? setSize: oldExtent andMakeResizable: outerMorph<br>
> ???????????????? | tw |<br>
> +?????????????? (tw := outerMorph findA: ScrollPane) ifNil: [^self].<br>
> -?????????????? (tw := outerMorph findA: TwoWayScrollPane) ifNil: [^self].<br>
> ???????????????? tw hResizing: #spaceFill;<br>
> ???????????????????????????????? vResizing: #spaceFill;<br>
> ???????????????????????????????? color: Color transparent;<br>
> ???????????????????????????????? setProperty: #hideUnneededScrollbars toValue: true.<br>
> ???????????????? outerMorph<br>
> ???????????????????????????????? hResizing: #shrinkWrap;<br>
> ???????????????????????????????? vResizing: #shrinkWrap;<br>
> ???????????????????????????????? cellPositioning: #topLeft.<br>
> ???????????????? outerMorph fullBounds.<br>
> ?? !<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: SyntaxMorph>>enclosingPane (in category 'accessing') -----<br>
> ?? enclosingPane<br>
> ???????????????? "The object that owns this script layout"<br>
> ??<br>
> ???????????????? | oo higher |<br>
> ???????????????? oo := self owner.<br>
> ???????????????? [higher := oo isSyntaxMorph.<br>
> ???????????????? higher := higher or: [oo class == TransformMorph].<br>
> +?????????????? higher := higher or: [oo class == ScrollPane].<br>
> -?????????????? higher := higher or: [oo class == TwoWayScrollPane].<br>
> ???????????????? higher ifFalse: [^ oo].<br>
> ???????????????? higher] whileTrue: [oo := oo owner].<br>
> ?? !<br>
> <br>
> Item was removed:<br>
> - ----- Method: SyntaxMorph>>inAScrollPane (in category 'initialization') -----<br>
> - inAScrollPane<br>
> -?????????????? "Answer a scroll pane in which the receiver is scrollable"<br>
> -<br>
> -?????????????? ^ self inATwoWayScrollPane!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: SyntaxMorph>>openInWindow (in category 'initialization') -----<br>
> ?? openInWindow<br>
> ??<br>
> +?????????????? | sel |<br>
> -?????????????? | window widget sel |<br>
> ???????????????? sel := ''.<br>
> ???????????????? self firstSubmorph allMorphs do: [:rr |<br>
> +?????????????????????????????? (rr isKindOf: StringMorph) ifTrue: [sel := sel, rr contents]].<br>
> -?????????????????????????????????????????????? (rr isKindOf: StringMorph) ifTrue: [sel := sel, rr contents]].<br>
> -?????????????? window := (SystemWindow labelled: 'Tiles for ', self parsedInClass printString, '>>',sel).<br>
> -?????????????? widget := self inAScrollPane.<br>
> -?????????????? widget color: Color paleOrange.<br>
> -?????????????? window<br>
> -?????????????????????????????? addMorph: widget<br>
> -?????????????????????????????? frame: (0@0 extent: 1.0@1.0).<br>
> -?????????????? window openInWorldExtent: (<br>
> -?????????????????????????????? self extent + (20@40) min: (Display boundingBox extent * 0.8) rounded<br>
> -?????????????? )<br>
> ??<br>
> +?????????????? ^ self inAScrollPane<br>
> +?????????????????????????????? color: Color paleOrange;<br>
> +?????????????????????????????? openInWindowLabeled: 'Tiles for ', self parsedInClass printString, '>>', sel!<br>
> - !<br>
> <br>
> Item was added:<br>
> + ----- Method: SyntaxMorph>>parseNodeWith:asStatement: (in category '*Etoys-Squeakland-code generation') -----<br>
> + parseNodeWith: encoder asStatement: aBoolean<br>
> +<br>
> +?????????????? ^ self parseNode!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: SyntaxMorph>>unhighlightBorder (in category 'highlighting') -----<br>
> ?? unhighlightBorder<br>
> ??<br>
> ???????????????? self currentSelectionDo: [:innerMorph :mouseDownLoc :outerMorph |<br>
> +?????????????????????????????? (self == outerMorph or: [owner notNil and: [owner isSyntaxMorph not]])<br>
> +?????????????????????????????????????????????? ifFalse: [self borderColor: self stdBorderColor]<br>
> +?????????????????????????????????????????????? ifTrue: [<br>
> +?????????????????????????????????????????????????????????????? (self hasProperty: #deselectedBorderColor)<br>
> +?????????????????????????????????????????????????????????????????????????????? ifTrue: [self borderColor: (self valueOfProperty: #deselectedBorderColor)]<br>
> +?????????????????????????????????????????????????????????????????????????????? ifFalse: [self borderStyle: (BorderStyle raised width: self borderWidth)]] ].!<br>
> -?????????????????????????????? self borderColor: (<br>
> -?????????????????????????????????????????????? (self == outerMorph or: [owner notNil and: [owner isSyntaxMorph not]])<br>
> -?????????????????????????????????????????????????????????????? ifFalse: [self borderColor: self stdBorderColor]<br>
> -?????????????????????????????????????????????????????????????? ifTrue: [<br>
> -?????????????????????????????????????????????????????????????????????????????? (self hasProperty: #deselectedBorderColor)<br>
> -?????????????????????????????????????????????????????????????????????????????????????????????? ifTrue: [self borderColor: (self valueOfProperty: #deselectedBorderColor)]<br>
> -?????????????????????????????????????????????????????????????????????????????????????????????? ifFalse: [self borderStyle: (BorderStyle raised width: self borderWidth)]] )].!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: TileMorph>>wrapPhraseInFunction (in category '*Etoys-Squeakland-arrows') -----<br>
> ?? wrapPhraseInFunction<br>
> ???????????????? "The user made a gesture requesting that the phrase for which the receiver bears the widget hit be wrapped in a function.?? This applies for the moment only to numeric functions"<br>
> ??<br>
> ???????????????? | pad newPad functionPhrase |<br>
> ???????????????? pad := self ownerThatIsA: TilePadMorph.?? "Or something higher than that???"<br>
> ???????????????? (pad isNil or: [pad type ~= #Number]) ifTrue: [^ Beeper beep].<br>
> ???????????????? newPad := TilePadMorph new setType: #Number.<br>
> +?????????????? newPad hResizing: #shrinkWrap; vResizing: #spaceFill.<br>
> -?????????????? newPad hResizing: #shrinkWrap; vResizing: #spacefill.<br>
> ???????????????? functionPhrase := FunctionTile new.<br>
> ???????????????? newPad addMorphBack: functionPhrase.<br>
> ???????????????? pad owner replaceSubmorph: pad by: newPad.<br>
> ???????????????? functionPhrase operator: #abs pad: pad.<br>
> ???????????????? functionPhrase addSuffixArrow.<br>
> ???????????????? self scriptEdited<br>
> ?? !<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: TilePadMorph>>wrapInFunction (in category '*Etoys-Squeakland-miscellaneous') -----<br>
> ?? wrapInFunction<br>
> ???????????????? "The user made a gesture requesting that the receiver be wrapped in a (numeric) function."<br>
> ??<br>
> ???????????????? | newPad functionPhrase |<br>
> ???????????????? newPad := TilePadMorph new setType: #Number.<br>
> +?????????????? newPad hResizing: #shrinkWrap; vResizing: #spaceFill.<br>
> -?????????????? newPad hResizing: #shrinkWrap; vResizing: #spacefill.<br>
> ???????????????? functionPhrase := FunctionTile new.<br>
> ???????????????? newPad addMorphBack: functionPhrase.<br>
> ???????????????? owner replaceSubmorph: self by: newPad.<br>
> ???????????????? functionPhrase operator: #abs pad: self.<br>
> ???????????????? self scriptEdited!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: VariableNode>>asMorphicSyntaxIn: (in category '*Etoys-tiles') -----<br>
> ?? asMorphicSyntaxIn: parent<br>
> ??<br>
> ???????????????? ^ parent addToken: self name<br>
> ???????????????????????????????????????????????? type: #variable<br>
> +?????????????????????????????????????????????? on: self shallowCopy?????? "don't hand out the prototype!! See VariableNode>>initialize"<br>
> -?????????????????????????????????????????????? on: self clone?? "don't hand out the prototype!! See VariableNode>>initialize"<br>
> ?? !<br>
> <br>
> Item was changed:<br>
> + ----- Method: WatchMorph class>>fontName:bgColor:centerColor: (in category 'instance creation') -----<br>
> - ----- Method: WatchMorph class>>fontName:bgColor:centerColor: (in category 'as yet unclassified') -----<br>
> ?? fontName: aString bgColor: aColor centerColor: otherColor<br>
> ???????????????? ^ self new<br>
> ???????????????????????????????? fontName: aString;<br>
> ???????????????????????????????? color: aColor;<br>
> ???????????????????????????????? centerColor: otherColor!<br>
> <br>
> Item was changed:<br>
> ?? ----- Method: WorldWindow class>>test2 (in category 'as yet unclassified') -----<br>
> ?? test2<br>
> ???????????????? "WorldWindow test2."<br>
> ??<br>
> ???????????????? | window world scrollPane |<br>
> ???????????????? world := WiWPasteUpMorph newWorldForProject: nil.<br>
> ???????????????? window := (WorldWindow labelled: 'Scrollable World') model: world.<br>
> +?????????????? window addMorph: (scrollPane := ScrollPane new model: world)<br>
> -?????????????? window addMorph: (scrollPane := TwoWayScrollPane new model: world)<br>
> ???????????????????????????????? frame: (0@0 extent: 1.0@1.0).<br>
> ???????????????? scrollPane scroller addMorph: world.<br>
> ???????????????? world hostWindow: window.<br>
> ???????????????? window openInWorld<br>
> ?? !<br>
> <br>
> <br>
> <br>
<br>
> <br>
<br>
<br>
</blockquote></div>