[squeak-dev] The Trunk: Tools-mt.910.mcz
Levente Uzonyi
leves at caesar.elte.hu
Tue Oct 29 15:00:43 UTC 2019
On Tue, 29 Oct 2019, commits at source.squeak.org wrote:
> Marcel Taeumel uploaded a new version of Tools to project The Trunk:
> http://source.squeak.org/trunk/Tools-mt.910.mcz
>
> ==================== Summary ====================
>
> Name: Tools-mt.910
> Author: mt
> Time: 29 October 2019, 12:39:01.998604 pm
> UUID: 981eb775-0636-d744-b7bf-8f13d72351b1
> Ancestors: Tools-mt.909
>
> Fixes (a regression in) list selection in change-list tool. It is not the job of the model to implement toggle in a multi-selection setting. The model only has to react to index selector and selection-list selectors.
>
> =============== Diff against Tools-mt.909 ===============
>
> Item was changed:
> ----- Method: ChangeList>>listSelectionAt:put: (in category 'viewing access') -----
> listSelectionAt: index put: value
>
> + listSelections at: index put: value.
> + self changed: #listIndex.!
> - ^ listSelections at: index put: value!
This breaks the convention that *at:put: returns the stored value.
Levente
>
> Item was changed:
> ----- Method: ChangeList>>toggleListIndex: (in category 'viewing access') -----
> toggleListIndex: newListIndex
>
> + listIndex = newListIndex ifTrue: [^ self].
> - listIndex ~= 0 ifTrue: [listSelections at: listIndex put: false].
> - newListIndex ~= 0 ifTrue: [listSelections at: newListIndex put: true].
> listIndex := newListIndex.
> +
> self changed: #listIndex.
> self contentsChanged!
More information about the Squeak-dev
mailing list
|