ToListElementEventHandler>>showHidePlaceholderIn: fixed

pharo-graphics · Oct 18, 2024 · 729d033 · 729d033
1 parent a676507
commit 729d033
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/Toplo-Widget-List/ToListElementEventHandler.class.st b/src/Toplo-Widget-List/ToListElementEventHandler.class.st
@@ -298,7 +298,9 @@ ToListElementEventHandler >> secondaryClickEvent: anEvent [
 { #category : #'list element handling' }
 ToListElementEventHandler >> showHidePlaceholderIn: aListElement [
 
-	aListElement dataAccessor ifEmpty: [ ^ aListElement showPlaceholder ].
+	aListElement dataAccessor ifEmpty: [ 
+		aListElement selecter deselectAll.
+		^ aListElement showPlaceholder ].
 	aListElement hidePlaceholder
 ]
 

diff --git a/src/Toplo-Widget-List/ToSievedListElementEventHandler.class.st b/src/Toplo-Widget-List/ToSievedListElementEventHandler.class.st
@@ -69,6 +69,8 @@ ToSievedListElementEventHandler >> listSelectionCommandAppliedEvent: aPreNotific
 	" first check if the request is for the primary selection model "
 	listSelModel = aPreNotification selectionModel ifFalse: [ ^ self ].
 
+	" have to do nothing if the list is empty. Else, the selection can be lost in case of single select"
+	listElement dataAccessor ifEmpty: [ ^ self ].
 	" reset the selection in case of single selection mode "
 	aPreNotification currentTarget isMultipleSelection ifFalse: [
 		sieveSelectionModel deselectAll ].