Merge pull request #1265 from demarey/table-alternate-row-colors

Let the presenter configure alternated raw colors at any time.
pharo-spec · Mar 25, 2022 · a49e12c · a49e12c
2 parents 17a4f79 + 982aab4
commit a49e12c
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 24 deletions.
diff --git a/src/Spec2-Adapters-Morphic/SpMorphicTableAdapter.class.st b/src/Spec2-Adapters-Morphic/SpMorphicTableAdapter.class.st
@@ -7,9 +7,6 @@ Is important notice here that most of the row/cell creation happens on SpMorphic
 Class {
 	#name : #SpMorphicTableAdapter,
 	#superclass : #SpAbstractMorphicListAdapter,
-	#instVars : [
-		'alternateRowColors'
-	],
 	#category : #'Spec2-Adapters-Morphic-Table'
 }
 
@@ -71,7 +68,7 @@ SpMorphicTableAdapter >> addModelTo: tableMorph [
 
 { #category : #configuring }
 SpMorphicTableAdapter >> alternateRowsColor [
-	alternateRowColors := true.
+	self widgetDo: [ :w | w alternateRowsColor: true ]
 ]
 
 { #category : #factory }
@@ -83,7 +80,6 @@ SpMorphicTableAdapter >> buildWidget [
 		intercellSpacing: self class intercellSpacing;
 		dragEnabled: self dragEnabled;
 		dropEnabled: self dropEnabled;
-		alternateRowsColor: alternateRowColors;
 		yourself.
 
 	self addModelTo: widget.
@@ -125,12 +121,6 @@ SpMorphicTableAdapter >> ifSorted: sortedBlock ifNotSorted: notSortedBlock [
 	^ notSortedBlock value
 ]
 
-{ #category : #initialization }
-SpMorphicTableAdapter >> initialize [ 
-	super initialize.
-	alternateRowColors := false.
-]
-
 { #category : #testing }
 SpMorphicTableAdapter >> isNonEditableRow: aRow column: aColumn [
 

diff --git a/src/Spec2-Tests/SpTablePresenterTest.class.st b/src/Spec2-Tests/SpTablePresenterTest.class.st
@@ -40,19 +40,6 @@ SpTablePresenterTest >> testAddColumnRaisesOneEventOnly [
 	self assert: count equals: 1
 ]
 
-{ #category : #tests }
-SpTablePresenterTest >> testCanAlternateRowColors [
-
-	presenter
-		hideColumnHeaders;
-		alternateRowsColor.
-
-	self openInstance.
-
-	presenter withAdapterPerformOrDefer: [ :adapter | 
-		self assert: (adapter instVarNamed: #alternateRowColors) equals: true ]
-]
-
 { #category : #tests }
 SpTablePresenterTest >> testGivenSortingIsActiveWhenAffectingNewItemsThenNewItemsAreSorted [