From 8207f15a839323404044cb91998a4353edc6608b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Carl-Gerhard=20Lindesva=CC=88rd?= <lindesvard@gmail.com>
Date: Mon, 22 Apr 2024 21:07:31 +0200
Subject: [PATCH] minor visual tweaks

---
 .../dashboards/[dashboardId]/list-reports.tsx |  9 +---
 .../profiles/profile-last-seen/index.tsx      | 49 ++++++++-----------
 2 files changed, 22 insertions(+), 36 deletions(-)

diff --git a/apps/dashboard/src/app/(app)/[organizationSlug]/[projectId]/dashboards/[dashboardId]/list-reports.tsx b/apps/dashboard/src/app/(app)/[organizationSlug]/[projectId]/dashboards/[dashboardId]/list-reports.tsx
index 5768a2f6..852d3409 100644
--- a/apps/dashboard/src/app/(app)/[organizationSlug]/[projectId]/dashboards/[dashboardId]/list-reports.tsx
+++ b/apps/dashboard/src/app/(app)/[organizationSlug]/[projectId]/dashboards/[dashboardId]/list-reports.tsx
@@ -62,7 +62,7 @@ export function ListReports({ reports }: ListReportsProps) {
           <span className="sm:hidden">Report</span>
         </Button>
       </StickyBelowHeader>
-      <div className="grid grid-cols-1 gap-4 p-4 sm:grid-cols-2">
+      <div className="mx-auto flex max-w-3xl flex-col gap-8 p-4 md:p-8">
         {reports.map((report) => {
           const chartRange = report.range; // timeRanges[report.range];
           return (
@@ -122,12 +122,7 @@ export function ListReports({ reports }: ListReportsProps) {
                   />
                 </div>
               </Link>
-              <div
-                className={cn(
-                  'p-4',
-                  report.chartType === 'bar' && 'max-h-[300px] overflow-auto'
-                )}
-              >
+              <div className={cn('p-4')}>
                 <LazyChart
                   {...report}
                   range={range ?? report.range}
diff --git a/apps/dashboard/src/app/(app)/[organizationSlug]/[projectId]/profiles/profile-last-seen/index.tsx b/apps/dashboard/src/app/(app)/[organizationSlug]/[projectId]/profiles/profile-last-seen/index.tsx
index 0c0cb4ce..c23a9ab5 100644
--- a/apps/dashboard/src/app/(app)/[organizationSlug]/[projectId]/profiles/profile-last-seen/index.tsx
+++ b/apps/dashboard/src/app/(app)/[organizationSlug]/[projectId]/profiles/profile-last-seen/index.tsx
@@ -21,54 +21,45 @@ export default async function ProfileLastSeenServer({ projectId }: Props) {
   // Days since last event from users
   // group by days
   const res = await chQuery<Row>(
-    `SELECT age('days',created_at, now()) as days, count(distinct profile_id) as count FROM events where project_id = ${escape(projectId)} group by days order by days ASC`
+    `SELECT age('days',created_at, now()) as days, count(distinct profile_id) as count FROM events where project_id = ${escape(projectId)} group by days order by days ASC LIMIT 51`
   );
 
-  const take = 18;
-  const split = take / 2;
-  const max = Math.max(...res.map((item) => item.count));
+  const maxValue = Math.max(...res.map((x) => x.count));
+  const minValue = Math.min(...res.map((x) => x.count));
+  const calculateRatio = (currentValue: number) =>
+    Math.max(
+      0.1,
+      Math.min(1, (currentValue - minValue) / (maxValue - minValue))
+    );
+
   const renderItem = (item: Row) => (
-    <div
-      key={item.days}
-      className="flex h-full flex-1 shrink-0 flex-col items-center"
-    >
+    <div className="flex w-1/12 flex-col items-center p-1">
       <Tooltip>
         <TooltipTrigger asChild>
-          <div className="flex w-full flex-1 flex-col justify-end rounded bg-slate-200">
-            <div
-              className={cn(
-                'w-full rounded',
-                item.days < split ? 'bg-blue-600' : 'bg-blue-400'
-              )}
-              style={{
-                height: `${(item.count / max) * 100}%`,
-              }}
-            />
-          </div>
+          <div
+            className={cn('aspect-square w-full shrink-0 rounded bg-blue-600')}
+            style={{
+              opacity: calculateRatio(item.count),
+            }}
+          ></div>
         </TooltipTrigger>
         <TooltipContent>
           {item.count} profiles last seen{' '}
           {item.days === 0 ? 'today' : `${item.days} days ago`}
         </TooltipContent>
       </Tooltip>
-      <div className="mt-1 text-xs">{item.days}</div>
+      <div className="mt-1 text-[10px]">{item.days}</div>
     </div>
   );
+
   return (
     <Widget className="w-full">
       <WidgetHead>
         <div className="title">Last seen</div>
       </WidgetHead>
       <WidgetBody>
-        <div className="flex aspect-[3/1] w-full items-end gap-1">
-          {res.length >= 18 ? (
-            <>
-              {res.slice(0, split).map(renderItem)}
-              {res.slice(-split).map(renderItem)}
-            </>
-          ) : (
-            res.map(renderItem)
-          )}
+        <div className="flex w-full flex-wrap items-start justify-start">
+          {res.map(renderItem)}
         </div>
         <div className="text-center text-xs text-muted-foreground">DAYS</div>
       </WidgetBody>