Skip to content

Commit

Permalink
Remove fast table search by default
Browse files Browse the repository at this point in the history
It often happens is a table or list has the focus that we start to type and a search popover appears but it filters nothing. And in multiple UIs, we have a filter done with an input field under the list and both are overlapping and it becomes really confusing.

I propose to disable the FastTable search by default and I also remove a dead method
  • Loading branch information
jecisc committed Jun 4, 2024
1 parent c05f2b3 commit 4c359a8
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 13 deletions.
12 changes: 0 additions & 12 deletions src/Spec2-Adapters-Morphic/SpAbstractMorphicListAdapter.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -135,18 +135,6 @@ SpAbstractMorphicListAdapter >> triggerActivationAction [
self presenter doActivateAtIndex: self presenter selection selectedIndex
]

{ #category : 'factory' }
SpAbstractMorphicListAdapter >> updateItemFilterBlockWith: block [

^ block
ifNotNil: [
widget
enableFilter: (SpFTSpecFilter block: block);
explicitFunction ]
ifNil: [
self updateSearch ]
]

{ #category : 'factory' }
SpAbstractMorphicListAdapter >> updateSearch [

Expand Down
2 changes: 1 addition & 1 deletion src/Spec2-Adapters-Morphic/SpMorphicListAdapter.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ SpMorphicListAdapter >> newTableWith: datasource [
^ SpFTTableMorph new
dataSource: datasource;
hideColumnHeaders;
enableSearch;
disableFunction;
beResizable;
columns: { self newListColumn };
setMultipleSelection: self model isMultipleSelection;
Expand Down
1 change: 1 addition & 0 deletions src/Spec2-Adapters-Morphic/SpMorphicTableAdapter.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,7 @@ SpMorphicTableAdapter >> buildWidget [
intercellSpacing: self class intercellSpacing;
dragEnabled: self dragEnabled;
dropEnabled: self dropEnabled;
disableFunction;
yourself.

self addModelTo: widget.
Expand Down

0 comments on commit 4c359a8

Please sign in to comment.