From ad87365a66ac84080a01cb374c0391a4a4532566 Mon Sep 17 00:00:00 2001
From: Liam <liam@fascinated.cc>
Date: Fri, 11 Oct 2024 19:58:49 +0100
Subject: [PATCH] cleanup

---
 .../src/components/player/player-header.tsx   | 29 +++++++++++++++++--
 1 file changed, 26 insertions(+), 3 deletions(-)

diff --git a/projects/website/src/components/player/player-header.tsx b/projects/website/src/components/player/player-header.tsx
index c620a15..9540a07 100644
--- a/projects/website/src/components/player/player-header.tsx
+++ b/projects/website/src/components/player/player-header.tsx
@@ -10,6 +10,7 @@ import { ReactElement } from "react";
 import PlayerTrackedStatus from "@/components/player/player-tracked-status";
 import ScoreSaberPlayer from "@ssr/common/types/player/impl/scoresaber-player";
 import Link from "next/link";
+import { capitalizeFirstLetter } from "@/common/string-utils";
 
 /**
  * Renders the change for a stat.
@@ -42,15 +43,37 @@ const renderChange = (player: ScoreSaberPlayer, type: "rank" | "countryRank" | "
   const todayStats = player.statisticChange?.today;
   const weeklyStats = player.statisticChange?.weekly;
   const monthlyStats = player.statisticChange?.monthly;
+  const todayStat = todayStats?.[type];
+  const weeklyStat = weeklyStats?.[type];
+  const monthlyStat = monthlyStats?.[type];
+
+  const renderChange = (value: number | undefined, timeFrame: "daily" | "weekly" | "monthly") => {
+    if (value == undefined) {
+      return "Missing Data";
+    }
+    const format = (value: number) => {
+      if (value == 0) {
+        return 0;
+      }
+      return type == "pp" ? formatPp(value) : formatNumberWithCommas(value);
+    };
+
+    return (
+      <p>
+        {capitalizeFirstLetter(timeFrame)} Change:{" "}
+        <span className={`${value >= 0 ? (value == 0 ? "" : "text-green-500") : "text-red-500"}`}>{format(value)}</span>
+      </p>
+    );
+  };
 
   return (
     <Tooltip
       side="bottom"
       display={
         <div>
-          <p>Daily Change: {todayStats?.[type] || "Missing Data"}</p>
-          <p>Weekly Change: {weeklyStats?.[type] || "Missing Data"}</p>
-          <p>Monthly Change: {monthlyStats?.[type] || "Missing Data"}</p>
+          {renderChange(todayStat, "daily")}
+          {renderChange(weeklyStat, "weekly")}
+          {renderChange(monthlyStat, "monthly")}
         </div>
       }
     >