From 59a2c49ab6ee5a1511a92074ee448b19d04ac1fa Mon Sep 17 00:00:00 2001
From: Vito0912 <86927734+Vito0912@users.noreply.github.com>
Date: Sat, 14 Dec 2024 18:54:14 +0100
Subject: [PATCH] fixed order and button style

---
 client/components/stats/YearInReviewBanner.vue | 6 +++---
 client/components/ui/Dropdown.vue              | 6 +++++-
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/client/components/stats/YearInReviewBanner.vue b/client/components/stats/YearInReviewBanner.vue
index 1ccaa9b0a..7bdaa564e 100644
--- a/client/components/stats/YearInReviewBanner.vue
+++ b/client/components/stats/YearInReviewBanner.vue
@@ -31,7 +31,7 @@
         <div class="flex-grow" />
 
         <!-- year selector -->
-        <ui-dropdown v-model="yearInReviewYear" small :items="availableYears()" class="m-6 text-sm mr-1 sm:mr-2 max-w-[100px]"/>
+        <ui-dropdown v-model="yearInReviewYear" small :items="availableYears()" v-if="availableYears().length >= 1" class="text-sm mr-1 sm:mr-2 max-w-[80px]" button-style="h-[29.55px] mt-[0.25px] !py-0 font-semibold"/>
 
         <!-- refresh button -->
         <ui-btn small :disabled="processingYearInReview" class="inline-flex items-center font-semibold mr-1 sm:mr-2" @click="refreshYearInReview">
@@ -70,7 +70,7 @@
           <div class="flex-grow" />
 
           <!-- year selector -->
-          <ui-dropdown v-model="yearInReviewYear" small :items="availableYears()" class="m-6 text-sm mr-1 sm:mr-2 max-w-[100px]"/>
+          <ui-dropdown v-model="yearInReviewYear" small :items="availableYears()" v-if="availableYears().length >= 1" class="text-sm mr-1 sm:mr-2 max-w-[80px]" button-style="h-[29.55px] mt-[0.25px] !py-0 font-semibold"/>
 
           <!-- refresh button -->
           <ui-btn small :disabled="processingYearInReviewServer" class="inline-flex items-center font-semibold mr-1 sm:mr-2" @click="refreshYearInReviewServer">
@@ -144,7 +144,7 @@ export default {
           const currentYear = new Date().getFullYear()
 
           const years = []
-          for (let year = oldestYear; year <= currentYear; year++) {
+          for (let year = currentYear; year >= oldestYear; year--) {
             years.push({ value: year, text: year.toString() })
           }
 
diff --git a/client/components/ui/Dropdown.vue b/client/components/ui/Dropdown.vue
index 3c142686f..1839e49aa 100644
--- a/client/components/ui/Dropdown.vue
+++ b/client/components/ui/Dropdown.vue
@@ -45,6 +45,9 @@ export default {
     menuMaxHeight: {
       type: String,
       default: '224px'
+    },
+    buttonStyle: {
+      type: String
     }
   },
   data() {
@@ -88,6 +91,7 @@ export default {
     },
     buttonClass() {
       var classes = []
+      if (this.buttonStyle) classes.push(this.buttonStyle)
       if (this.small) classes.push('h-9')
       else classes.push('h-10')
 
@@ -119,4 +123,4 @@ export default {
   },
   mounted() {}
 }
-</script>
\ No newline at end of file
+</script>