Merge pull request #11094 from demarey/move-over-color-to-theme

Specify the mouse over color in the theme rather than hardcode it.
pharo-project · Apr 4, 2022 · bfb3a61 · bfb3a61
2 parents 7784e0c + caf1b81
commit bfb3a61
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 1 deletion.
diff --git a/src/Morphic-Widgets-FastTable/FTSelectableMorph.class.st b/src/Morphic-Widgets-FastTable/FTSelectableMorph.class.st
@@ -43,7 +43,7 @@ FTSelectableMorph >> mouseLeave: evt [
 
 { #category : #accessing }
 FTSelectableMorph >> mouseOverColor [
-	^ self theme backgroundColor darker darker
+	^ self theme mouseOverColor
 ]
 
 { #category : #initialization }

diff --git a/src/Polymorph-Widgets/PharoDarkTheme.class.st b/src/Polymorph-Widgets/PharoDarkTheme.class.st
@@ -341,6 +341,11 @@ PharoDarkTheme >> menuItemDisabledTextColorFor: aMenuItem [
 	^ Color lightGray
 ]
 
+{ #category : #'accessing - colors' }
+PharoDarkTheme >> mouseOverColor [ 
+	^ self lightBackgroundColor
+]
+
 { #category : #initialization }
 PharoDarkTheme >> newRadioMarkerForm [
 	"Answer a new checkbox marker form."

diff --git a/src/Polymorph-Widgets/UITheme.class.st b/src/Polymorph-Widgets/UITheme.class.st
@@ -2752,6 +2752,15 @@ UITheme >> morphTreeSplitterPressedFillStyleFor: aSplitter [
 		radial: false
 ]
 
+{ #category : #'accessing - colors' }
+UITheme >> mouseOverColor [ 
+	"This color is used when hovering a raw of a list / table.
+	This color should be different from the default background color and from the raw alternate color
+	(see FTTableContainerMorph>>#alternateRowsColor."
+
+	^ self backgroundColor darker darker
+]
+
 { #category : #'fill-styles' }
 UITheme >> multiFormFillStyleFrom: anArray cornerWidths: widthArray in: aRectangle [
 	"Answer a composite fill style from the given forms and central colour