From ee6579bc327223741d1bc0cc8dd8ca98a2673aa2 Mon Sep 17 00:00:00 2001
From: adibarra <93070681+adibarra@users.noreply.github.com>
Date: Fri, 20 Mar 2026 01:31:32 -0500
Subject: [PATCH] move date picker quick select buttons to footer

---
 .../src/components/ui/date-range-picker.tsx   | 149 ++++++++----------
 1 file changed, 70 insertions(+), 79 deletions(-)

diff --git a/packages/app/src/components/ui/date-range-picker.tsx b/packages/app/src/components/ui/date-range-picker.tsx
index 751acef..8cfd27b 100644
--- a/packages/app/src/components/ui/date-range-picker.tsx
+++ b/packages/app/src/components/ui/date-range-picker.tsx
@@ -189,69 +189,6 @@ export function DateRangePicker({
               )}
             </DialogDescription>
           </DialogHeader>
-          {availableDates && availableDates.length >= 2 && (
-            <div className="flex flex-wrap gap-2">
-              {[
-                {
-                  label: 'Max Range',
-                  getRange: () => ({
-                    startDate: availableDates[0],
-                    endDate: availableDates[availableDates.length - 1],
-                  }),
-                },
-                {
-                  label: 'Last 30 Days',
-                  getRange: () => {
-                    const cutoff = new Date();
-                    cutoff.setDate(cutoff.getDate() - 30);
-                    const cutoffStr = cutoff.toISOString().slice(0, 10);
-                    const filtered = availableDates.filter((d) => d >= cutoffStr);
-                    if (filtered.length < 2) return null;
-                    return { startDate: filtered[0], endDate: filtered[filtered.length - 1] };
-                  },
-                },
-                {
-                  label: 'Last 60 Days',
-                  getRange: () => {
-                    const cutoff = new Date();
-                    cutoff.setDate(cutoff.getDate() - 60);
-                    const cutoffStr = cutoff.toISOString().slice(0, 10);
-                    const filtered = availableDates.filter((d) => d >= cutoffStr);
-                    if (filtered.length < 2) return null;
-                    return { startDate: filtered[0], endDate: filtered[filtered.length - 1] };
-                  },
-                },
-                {
-                  label: 'Last 90 Days',
-                  getRange: () => {
-                    const cutoff = new Date();
-                    cutoff.setDate(cutoff.getDate() - 90);
-                    const cutoffStr = cutoff.toISOString().slice(0, 10);
-                    const filtered = availableDates.filter((d) => d >= cutoffStr);
-                    if (filtered.length < 2) return null;
-                    return { startDate: filtered[0], endDate: filtered[filtered.length - 1] };
-                  },
-                },
-              ].map(({ label, getRange }) => {
-                const range = getRange();
-                if (!range) return null;
-                return (
-                  <Button
-                    key={label}
-                    variant="outline"
-                    size="sm"
-                    className="text-xs h-7"
-                    onClick={() => {
-                      setTempRange(range);
-                      track('date_range_picker_quick_select', { label });
-                    }}
-                  >
-                    {label}
-                  </Button>
-                );
-              })}
-            </div>
-          )}
           <div className="py-4 relative">
             <CalendarGrid
               dateRange={tempRange}
@@ -299,23 +236,77 @@ export function DateRangePicker({
               )}
           </div>
           {error && <p className="text-md text-center text-red-500">{error}</p>}
-          <DialogFooter>
-            <DialogClose asChild>
-              <Button variant="outline" onClick={handleCancel}>
-                Cancel
+          <DialogFooter className="flex-row justify-between sm:justify-between">
+            {availableDates && availableDates.length >= 2 ? (
+              <div className="flex flex-wrap gap-1.5">
+                {[
+                  {
+                    label: 'Max Range',
+                    getRange: () => ({
+                      startDate: availableDates[0],
+                      endDate: availableDates[availableDates.length - 1],
+                    }),
+                  },
+                  {
+                    label: 'Last 90 Days',
+                    getRange: () => {
+                      const cutoff = new Date();
+                      cutoff.setDate(cutoff.getDate() - 90);
+                      const cutoffStr = cutoff.toISOString().slice(0, 10);
+                      const filtered = availableDates.filter((d) => d >= cutoffStr);
+                      if (filtered.length < 2) return null;
+                      return { startDate: filtered[0], endDate: filtered[filtered.length - 1] };
+                    },
+                  },
+                  {
+                    label: 'Last 30 Days',
+                    getRange: () => {
+                      const cutoff = new Date();
+                      cutoff.setDate(cutoff.getDate() - 30);
+                      const cutoffStr = cutoff.toISOString().slice(0, 10);
+                      const filtered = availableDates.filter((d) => d >= cutoffStr);
+                      if (filtered.length < 2) return null;
+                      return { startDate: filtered[0], endDate: filtered[filtered.length - 1] };
+                    },
+                  },
+                ].map(({ label, getRange }) => {
+                  const range = getRange();
+                  if (!range) return null;
+                  return (
+                    <Button
+                      key={label}
+                      variant="outline"
+                      onClick={() => {
+                        setTempRange(range);
+                        track('date_range_picker_quick_select', { label });
+                      }}
+                    >
+                      {label}
+                    </Button>
+                  );
+                })}
+              </div>
+            ) : (
+              <div />
+            )}
+            <div className="flex gap-2">
+              <DialogClose asChild>
+                <Button variant="outline" onClick={handleCancel}>
+                  Cancel
+                </Button>
+              </DialogClose>
+              <Button
+                onClick={handleApply}
+                disabled={
+                  !tempRange.startDate ||
+                  !tempRange.endDate ||
+                  isApplying ||
+                  (availableDates !== undefined && availableDates.length < 2)
+                }
+              >
+                {isApplying ? 'Applying...' : 'Apply'}
               </Button>
-            </DialogClose>
-            <Button
-              onClick={handleApply}
-              disabled={
-                !tempRange.startDate ||
-                !tempRange.endDate ||
-                isApplying ||
-                (availableDates !== undefined && availableDates.length < 2)
-              }
-            >
-              {isApplying ? 'Applying...' : 'Apply'}
-            </Button>
+            </div>
           </DialogFooter>
         </DialogContent>
       </Dialog>