search now works correctly!

ui/browsingpane.go

@@ -6,6 +6,8 @@ import (
 
 	"fyne.io/fyne/v2"
 	"fyne.io/fyne/v2/container"
+	"fyne.io/fyne/v2/layout"
+	"fyne.io/fyne/v2/theme"
 	"fyne.io/fyne/v2/widget"
 )
 
@@ -21,8 +23,7 @@ type BrowsingPane struct {
 	widget.BaseWidget
 
 	searchBar           *searchField
-	searchTimer         time.Timer
-	pendingSearch       bool
+	searchTimer         *time.Timer
 	cancelPendingSearch context.CancelFunc
 	curPage             Page
 
@@ -36,7 +37,49 @@ type blankPage struct {
 
 type searchField struct {
 	widget.Entry
-	height float32
+	height        float32
+	OnTextChanged func(string)
+}
+
+var _ fyne.Tappable = (*clearTextButton)(nil)
+
+type clearTextButton struct {
+	widget.Icon
+
+	OnTapped func()
+}
+
+type hspace struct {
+	widget.BaseWidget
+
+	Width float32
+}
+
+func newHSpace(w float32) *hspace {
+	h := &hspace{Width: w}
+	h.ExtendBaseWidget(h)
+	return h
+}
+
+func (h *hspace) MinSize() fyne.Size {
+	return fyne.NewSize(h.Width, 0)
+}
+
+func (h *hspace) CreateRenderer() fyne.WidgetRenderer {
+	return widget.NewSimpleRenderer(layout.NewSpacer())
+}
+
+func NewClearTextButton() *clearTextButton {
+	c := &clearTextButton{}
+	c.ExtendBaseWidget(c)
+	c.Resource = theme.SearchIcon()
+	return c
+}
+
+func (c *clearTextButton) Tapped(*fyne.PointEvent) {
+	if c.OnTapped != nil {
+		c.OnTapped()
+	}
 }
 
 func NewSearchField() *searchField {
@@ -45,6 +88,22 @@ func NewSearchField() *searchField {
 	// this is a bit hacky
 	sf.height = widget.NewEntry().MinSize().Height
 	sf.PlaceHolder = "Search"
+	c := NewClearTextButton()
+	c.OnTapped = func() {
+		sf.SetText("")
+	}
+	sf.ActionItem = c
+	sf.OnChanged = func(s string) {
+		if s == "" {
+			c.Resource = theme.SearchIcon()
+		} else {
+			c.Resource = theme.ContentClearIcon()
+		}
+		c.Refresh()
+		if sf.OnTextChanged != nil {
+			sf.OnTextChanged(s)
+		}
+	}
 	return sf
 }
 
@@ -56,11 +115,12 @@ func NewBrowsingPane() *BrowsingPane {
 	b := &BrowsingPane{}
 	b.ExtendBaseWidget(b)
 	b.searchBar = NewSearchField()
-	b.searchBar.OnChanged = b.onSearchTextChanged
+	b.searchBar.OnTextChanged = b.onSearchTextChanged
 	b.curPage = &blankPage{}
-	b.searchTimer = *time.NewTimer(0)
 	b.pageContaner = container.NewMax(b.curPage)
-	b.container = container.NewBorder(container.NewHBox(b.searchBar), nil, nil, nil, b.pageContaner)
+	b.container = container.NewBorder(
+		container.NewHBox(newHSpace(15), b.searchBar),
+		nil, nil, nil, b.pageContaner)
 	return b
 }
 
@@ -75,30 +135,46 @@ func (b *BrowsingPane) SetPage(p Page) {
 }
 
 func (b *BrowsingPane) onSearchTextChanged(text string) {
-	b.searchTimer.Reset(300 * time.Millisecond)
-	if s, ok := b.curPage.(Searchable); ok {
-		s.OnSearched(text)
+	if text == "" {
+		if b.cancelPendingSearch != nil {
+			b.cancelPendingSearch()
+		}
+		b.sendSearch("")
 	}
-	if !b.pendingSearch {
-		/*
-			ctx, cancel := context.WithCancel(context.Background())
-			b.cancelPendingSearch = cancel
-			b.pendingSearch = true
-			go func(ctx context.Context) {
-				select {
-				case <-ctx.Done():
-					b.pendingSearch = false
-				case <-b.searchTimer.C:
-					if s, ok := b.curPage.(Searchable); ok {
-						s.OnSearched(b.searchBar.Text)
-					}
-					b.pendingSearch = false
-				}
-			}(ctx)
-		*/
+	if b.searchTimer == nil {
+		ctx, cancel := context.WithCancel(context.Background())
+		b.cancelPendingSearch = cancel
+		b.searchTimer = time.NewTimer(200 * time.Millisecond)
+		go func(ctx context.Context, trigger <-chan time.Time) {
+			select {
+			case <-ctx.Done():
+				b.searchTimer = nil
+			case <-trigger:
+				b.sendSearch(b.searchBar.Text)
+				b.searchTimer = nil
+			}
+		}(ctx, b.searchTimer.C)
+	} else {
+		b.resetSearchTimer(200 * time.Millisecond)
+	}
+}
+
+func (b *BrowsingPane) sendSearch(query string) {
+	if s, ok := b.curPage.(Searchable); ok {
+		s.OnSearched(query)
 	}
 }
 
 func (b *BrowsingPane) CreateRenderer() fyne.WidgetRenderer {
 	return widget.NewSimpleRenderer(b.container)
 }
+
+func (b *BrowsingPane) resetSearchTimer(d time.Duration) {
+	if b.searchTimer == nil {
+		return
+	}
+	if !b.searchTimer.Stop() {
+		<-b.searchTimer.C
+	}
+	b.searchTimer.Reset(d)
+}