On Mon, 28 Feb 2011, commits@source.squeak.org wrote:
A new version of Morphic was added to project The Inbox: http://source.squeak.org/inbox/Morphic-cao.522.mcz
==================== Summary ====================
Name: Morphic-cao.522 Author: cao Time: 28 February 2011, 6:22:34.55 pm UUID: 4be5ac71-d5df-4e96-b59c-5065ad7932f9 Ancestors: Morphic-mtf.521
StringMorph>>contents: shouldn't mutate fonts in the test attributes.
Can you provide a simple test case that shows the effect of this change for someone who's not familiar with the code (like me)?
Levente
=============== Diff against Morphic-mtf.521 ===============
Item was changed: ----- Method: StringMorph>>contents: (in category 'accessing') ----- contents: newContents | scanner | contents := newContents isText ifTrue: [scanner := StringMorphAttributeScanner new initializeFromStringMorph: self. (newContents attributesAt: 1 forStyle: self font textStyle) do: [:attr | attr emphasizeScanner: scanner]. emphasis := scanner emphasis.
font := scanner font.
self fitContents!font := scanner font emphasis: emphasis. color := scanner textColor. newContents string] ifFalse: [contents = newContents ifTrue: [^ self]. "no substantive change" newContents].
squeak-dev@lists.squeakfoundation.org