add option to show album years in grid views

backend/config.go

@@ -59,6 +59,7 @@ type AlbumPageConfig struct {
 
 type AlbumsPageConfig struct {
 	SortOrder string
+	ShowYears bool
 }
 
 type ArtistPageConfig struct {
@@ -73,6 +74,7 @@ type ArtistsPageConfig struct {
 type FavoritesPageConfig struct {
 	InitialView      string
 	TracklistColumns []string
+	ShowAlbumYears   bool
 }
 
 type PlaylistPageConfig struct {
@@ -167,6 +169,7 @@ func DefaultConfig(appVersionTag string) *Config {
 		},
 		AlbumsPage: AlbumsPageConfig{
 			SortOrder: string("Recently Added"),
+			ShowYears: false,
 		},
 		ArtistPage: ArtistPageConfig{
 			InitialView:      "Discography",
@@ -178,6 +181,7 @@ func DefaultConfig(appVersionTag string) *Config {
 		FavoritesPage: FavoritesPageConfig{
 			TracklistColumns: []string{"Album", "Time", "Plays"},
 			InitialView:      "Albums",
+			ShowAlbumYears:   false,
 		},
 		PlaylistPage: PlaylistPageConfig{
 			TracklistColumns: []string{"Album", "Time", "Plays"},

ui/browsing/albumspage.go

@@ -72,6 +72,12 @@ func (a *albumsPageAdapter) SearchIter(query string, filter mediaprovider.AlbumF
 	return widgets.NewGridViewAlbumIterator(a.mp.SearchAlbums(query, filter))
 }
 
-func (a *albumsPageAdapter) ConnectGridActions(gv *widgets.GridView) {
+func (a *albumsPageAdapter) InitGrid(gv *widgets.GridView) {
 	a.contr.ConnectAlbumGridActions(gv)
+	gv.ShowSuffix = a.cfg.ShowYears
+}
+
+func (a *albumsPageAdapter) RefreshGrid(gv *widgets.GridView) {
+	gv.ShowSuffix = a.cfg.ShowYears
+	gv.Refresh()
 }

ui/browsing/artistspage.go

@@ -68,11 +68,15 @@ func (a *artistsPageAdapter) SearchIter(query string, filter mediaprovider.Artis
 	return widgets.NewGridViewArtistIterator(a.mp.SearchArtists(query, filter))
 }
 
-func (a *artistsPageAdapter) ConnectGridActions(gv *widgets.GridView) {
+func (a *artistsPageAdapter) InitGrid(gv *widgets.GridView) {
 	canShareArtists := false
 	if r, canShare := a.mp.(mediaprovider.SupportsSharing); canShare {
 		canShareArtists = r.CanShareArtists()
 	}
 	gv.DisableSharing = !canShareArtists
 	a.contr.ConnectArtistGridActions(gv)
 }
+
+func (a *artistsPageAdapter) RefreshGrid(gv *widgets.GridView) {
+	gv.Refresh()
+}

ui/browsing/browsingpane.go

@@ -195,6 +195,12 @@ func (b *BrowsingPane) ScrollDown() {
 	}
 }
 
+func (b *BrowsingPane) RefreshPage() {
+	if b.curPage != nil {
+		b.curPage.Refresh()
+	}
+}
+
 func (b *BrowsingPane) doSetPage(p Page) bool {
 	if b.curPage != nil && b.curPage.Route() == p.Route() {
 		return false

ui/browsing/favoritespage.go

@@ -70,6 +70,7 @@ func NewFavoritesPage(cfg *backend.FavoritesPageConfig, pool *util.WidgetPool, c
 	} else {
 		a.albumGrid = widgets.NewGridView(iter, a.im, myTheme.AlbumIcon)
 	}
+	a.albumGrid.ShowSuffix = cfg.ShowAlbumYears
 	a.contr.ConnectAlbumGridActions(a.albumGrid)
 	if cfg.InitialView == "Artists" {
 		a.toggleBtns.SetActivatedButton(1)
@@ -285,6 +286,13 @@ func (a *FavoritesPage) SelectAll() {
 	}
 }
 
+func (a *FavoritesPage) Refresh() {
+	if a.albumGrid != nil {
+		a.albumGrid.ShowSuffix = a.cfg.ShowAlbumYears
+	}
+	a.BaseWidget.Refresh()
+}
+
 func (a *FavoritesPage) tracklistOrNil() *widgets.Tracklist {
 	if a.tracklistCtr != nil {
 		return a.tracklistCtr.Objects[0].(*widgets.Tracklist)

ui/browsing/genrepage.go

@@ -14,15 +14,16 @@ import (
 
 type genrePageAdapter struct {
 	genre     string
+	cfg       *backend.AlbumsPageConfig
 	contr     *controller.Controller
 	mp        mediaprovider.MediaProvider
 	pm        *backend.PlaybackManager
 	filter    mediaprovider.AlbumFilter
 	filterBtn *widgets.AlbumFilterButton
 }
 
-func NewGenrePage(genre string, pool *util.WidgetPool, contr *controller.Controller, pm *backend.PlaybackManager, mp mediaprovider.MediaProvider, im *backend.ImageManager) Page {
-	adapter := &genrePageAdapter{genre: genre, contr: contr, mp: mp, pm: pm}
+func NewGenrePage(genre string, cfg *backend.AlbumsPageConfig, pool *util.WidgetPool, contr *controller.Controller, pm *backend.PlaybackManager, mp mediaprovider.MediaProvider, im *backend.ImageManager) Page {
+	adapter := &genrePageAdapter{genre: genre, cfg: cfg, contr: contr, mp: mp, pm: pm}
 	return NewGridViewPage(adapter, pool, mp, im)
 }
 
@@ -68,6 +69,12 @@ func (a *genrePageAdapter) SearchIter(query string, filter mediaprovider.AlbumFi
 	return widgets.NewGridViewAlbumIterator(a.mp.SearchAlbums(query, filter))
 }
 
-func (g *genrePageAdapter) ConnectGridActions(gv *widgets.GridView) {
+func (g *genrePageAdapter) InitGrid(gv *widgets.GridView) {
 	g.contr.ConnectAlbumGridActions(gv)
+	gv.ShowSuffix = g.cfg.ShowYears
+}
+
+func (g *genrePageAdapter) RefreshGrid(gv *widgets.GridView) {
+	gv.ShowSuffix = g.cfg.ShowYears
+	gv.Refresh()
 }

ui/browsing/gridviewpage.go

@@ -66,8 +66,13 @@ type GridViewPageAdapter[M, F any] interface {
 	// Returns the iterator for the given search query and filter.
 	SearchIter(query string, filter mediaprovider.MediaFilter[M, F]) widgets.GridViewIterator
 
-	// Function that connects the GridView callbacks to the appropriate action handlers.
-	ConnectGridActions(*widgets.GridView)
+	// Function that initialized the GridView with page-specific settings
+	// and connects the GridView callbacks to the appropriate action handlers.
+	InitGrid(*widgets.GridView)
+
+	// Function called when settings may have changed and the grid needs
+	// reconfiguring with possible settings changes.
+	RefreshGrid(*widgets.GridView)
 }
 
 type SortableGridViewPageAdapter interface {
@@ -124,7 +129,7 @@ func NewGridViewPage[M, F any](
 		gp.grid = widgets.NewGridView(iter, im, adapter.PlaceholderResource())
 	}
 	gp.grid.DisableSharing = !canShare
-	adapter.ConnectGridActions(gp.grid)
+	adapter.InitGrid(gp.grid)
 	gp.createSearchAndFilter()
 	gp.createContainer()
 	return gp
@@ -206,6 +211,10 @@ func (g *GridViewPage[M, F]) OnSearched(query string) {
 	g.searchText = query
 }
 
+func (g *GridViewPage[M, F]) Refresh() {
+	g.adapter.RefreshGrid(g.grid)
+}
+
 func (g *GridViewPage[M, F]) doSearch(query string) {
 	if g.searchText == "" {
 		g.gridState = g.grid.SaveToState()
@@ -294,7 +303,7 @@ func (s *savedGridViewPage[M, F]) Restore() Page {
 	} else {
 		gp.grid = widgets.NewGridViewFromState(state)
 	}
-	gp.adapter.ConnectGridActions(gp.grid)
+	gp.adapter.InitGrid(gp.grid)
 	gp.createSearchAndFilter()
 	gp.createContainer()
 	return gp

ui/browsing/router.go

@@ -44,7 +44,7 @@ func (r Router) CreatePage(rte controller.Route) Page {
 	case controller.Favorites:
 		return NewFavoritesPage(&r.App.Config.FavoritesPage, r.widgetPool, r.Controller, r.App.ServerManager.Server, r.App.PlaybackManager, r.App.ImageManager)
 	case controller.Genre:
-		return NewGenrePage(rte.Arg, r.widgetPool, r.Controller, r.App.PlaybackManager, r.App.ServerManager.Server, r.App.ImageManager)
+		return NewGenrePage(rte.Arg, &r.App.Config.AlbumsPage, r.widgetPool, r.Controller, r.App.PlaybackManager, r.App.ServerManager.Server, r.App.ImageManager)
 	case controller.Genres:
 		return NewGenresPage(r.Controller, r.App.ServerManager.Server)
 	case controller.NowPlaying:

ui/controller/controller.go

@@ -32,17 +32,16 @@ import (
 
 type NavigationHandler func(Route)
 
-type ReloadFunc func()
-
 type CurPageFunc func() Route
 
 type Controller struct {
-	AppVersion  string
-	MainWindow  fyne.Window
-	App         *backend.App
-	NavHandler  NavigationHandler
-	CurPageFunc CurPageFunc
-	ReloadFunc  ReloadFunc
+	AppVersion      string
+	App             *backend.App
+	MainWindow      fyne.Window
+	NavHandler      NavigationHandler
+	CurPageFunc     CurPageFunc
+	ReloadFunc      func()
+	RefreshPageFunc func()
 
 	escapablePopUp   *widget.PopUp
 	haveModal        bool
@@ -559,6 +558,7 @@ func (c *Controller) ShowSettingsDialog(themeUpdateCallbk func(), themeFiles map
 		copy(eq.BandGains[:], c.App.Config.LocalPlayback.GraphicEqualizerBands)
 		c.App.LocalPlayer.SetEqualizer(eq)
 	}
+	dlg.OnPageNeedsRefresh = c.RefreshPageFunc
 	pop := widget.NewModalPopUp(dlg, c.MainWindow.Canvas())
 	dlg.OnDismiss = func() {
 		pop.Hide()

ui/dialogs/settingsdialog.go

@@ -34,6 +34,7 @@ type SettingsDialog struct {
 	OnThemeSettingChanged          func()
 	OnDismiss                      func()
 	OnEqualizerSettingsChanged     func()
+	OnPageNeedsRefresh             func()
 
 	config       *backend.Config
 	audioDevices []mpv.AudioDevice
@@ -185,6 +186,14 @@ func (s *SettingsDialog) createGeneralTab(canSaveQueueToServer bool) *container.
 
 	trackNotif := widget.NewCheckWithData("Show notification on track change",
 		binding.BindBool(&s.config.Application.ShowTrackChangeNotification))
+	albumGridYears := widget.NewCheck("Show year in album grid cards", func(b bool) {
+		s.config.AlbumsPage.ShowYears = b
+		s.config.FavoritesPage.ShowAlbumYears = b
+		if s.OnPageNeedsRefresh != nil {
+			s.OnPageNeedsRefresh()
+		}
+	})
+	albumGridYears.Checked = s.config.AlbumsPage.ShowYears
 
 	// Scrobble settings
 
@@ -276,6 +285,7 @@ func (s *SettingsDialog) createGeneralTab(canSaveQueueToServer bool) *container.
 		container.NewHBox(systemTrayEnable, closeToTray),
 		saveQueueHBox,
 		trackNotif,
+		albumGridYears,
 		s.newSectionSeparator(),
 
 		widget.NewRichText(&widget.TextSegment{Text: "Scrobbling", Style: util.BoldRichTextStyle}),

ui/mainwindow.go

@@ -85,6 +85,7 @@ func NewMainWindow(fyneApp fyne.App, appName, displayAppName, appVersion string,
 	m.Controller.NavHandler = m.Router.NavigateTo
 	m.Controller.ReloadFunc = m.BrowsingPane.Reload
 	m.Controller.CurPageFunc = m.BrowsingPane.CurrentPage
+	m.Controller.RefreshPageFunc = m.BrowsingPane.RefreshPage
 
 	m.BottomPanel = NewBottomPanel(app.PlaybackManager, app.ImageManager, m.Controller)
 	m.container = container.NewBorder(nil, m.BottomPanel, nil, nil, m.BrowsingPane)

ui/theme/theme.go

@@ -26,6 +26,9 @@ const (
 	ColorNameIconButton        fyne.ThemeColorName = "IconButton"
 	ColorNameHoveredIconButton fyne.ThemeColorName = "HoveredIconButton"
 	ColorNameNowPlayingPanel   fyne.ThemeColorName = "NowPlayingPanel"
+
+	SizeNameSubText    fyne.ThemeSizeName = "subText"    // in between Text and Caption
+	SizeNameSuffixText fyne.ThemeSizeName = "suffixText" // a tiny bit smaller than subText
 )
 
 var (
@@ -243,6 +246,11 @@ func (m *MyTheme) Font(style fyne.TextStyle) fyne.Resource {
 }
 
 func (m *MyTheme) Size(name fyne.ThemeSizeName) float32 {
+	if name == SizeNameSubText {
+		return 13
+	} else if name == SizeNameSuffixText {
+		return 12
+	}
 	return theme.DefaultTheme().Size(name)
 }
 

ui/widgets/gridview.go

@@ -3,6 +3,7 @@ package widgets
 import (
 	"context"
 	"fmt"
+	"strconv"
 	"sync"
 
 	"github.com/dweymouth/supersonic/backend/mediaprovider"
@@ -51,13 +52,17 @@ type gridViewAlbumIterator struct {
 func (g gridViewAlbumIterator) NextN(n int) []GridViewItemModel {
 	albums := g.iter.NextN(n)
 	return sharedutil.MapSlice(albums, func(al *mediaprovider.Album) GridViewItemModel {
-		return GridViewItemModel{
+		model := GridViewItemModel{
 			Name:         al.Name,
 			ID:           al.ID,
 			CoverArtID:   al.CoverArtID,
 			Secondary:    al.ArtistNames,
 			SecondaryIDs: al.ArtistIDs,
 		}
+		if al.Year > 0 {
+			model.Suffix = strconv.Itoa(al.Year)
+		}
+		return model
 	})
 }
 
@@ -92,6 +97,8 @@ func NewGridViewArtistIterator(iter mediaprovider.ArtistIterator) GridViewIterat
 type GridView struct {
 	widget.BaseWidget
 
+	ShowSuffix bool
+
 	stateMutex  sync.RWMutex
 	fetchCancel context.CancelFunc
 	GridViewState
@@ -322,9 +329,11 @@ func (g *GridView) doUpdateItemCard(itemIdx int, card *GridViewItem) {
 	card.ItemIndex = itemIdx
 	g.itemForIndex[itemIdx] = card
 	card.Cover.Im.PlaceholderIcon = g.Placeholder
+	card.ShowSuffix = g.ShowSuffix
 	if !card.NeedsUpdate(item) && card.ItemIndex == itemIdx {
 		// nothing to do
 		g.stateMutex.Unlock()
+		card.Refresh()
 		return
 	}
 	g.stateMutex.Unlock()