From a055884d5c6c59d2bd4894a1b5514ab04168e10d Mon Sep 17 00:00:00 2001
From: Mathijs Bernson <mathijs@bernson.eu>
Date: Tue, 7 Jul 2026 20:34:53 +0200
Subject: [PATCH] Fix spacing in year picker

Fixes #40
---
 HackerTube/Features/Browse/BrowseView.swift | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/HackerTube/Features/Browse/BrowseView.swift b/HackerTube/Features/Browse/BrowseView.swift
index 8d31eaa..96dbd24 100644
--- a/HackerTube/Features/Browse/BrowseView.swift
+++ b/HackerTube/Features/Browse/BrowseView.swift
@@ -36,6 +36,7 @@ struct BrowseView: View {
                 #if os(tvOS)
                     if query == .popular {
                         YearPicker(year: $year)
+                            .pickerStyle(.menu)
                     }
                 #endif
 
@@ -51,7 +52,11 @@ struct BrowseView: View {
                         #endif
 
                         ToolbarItem(placement: .topBarTrailing) {
-                            YearPicker(year: $year)
+                            Menu {
+                                YearPicker(year: $year)
+                            } label: {
+                                Label("Year", systemImage: "calendar")
+                            }
                         }
                     }
                 #endif
@@ -93,21 +98,17 @@ struct BrowseView: View {
     }
 }
 
-struct YearPicker: View {
+private struct YearPicker: View {
     @Binding var year: Int
     let firstYear: Int = 2000
     let currentYear = Calendar.current.component(.year, from: .now)
     var body: some View {
-        Picker(selection: $year) {
+        Picker("Year", selection: $year) {
             ForEach(Array(firstYear...currentYear).reversed(), id: \.self) { year in
-                Text(String(year))
+                Text(year, format: .number.grouping(.never))
                     .tag(year)
             }
-        } label: {
-            Label("Year", systemImage: "calendar")
         }
-        .pickerStyle(.menu)
-        .fixedSize(horizontal: true, vertical: false)
     }
 }