fix(dashboard): polish revenue chart issues found in UI smoke

Three fixes from screenshotting the chart in every granularity:

- Sub-thousand bar labels showed a pointless decimal (€903.6) and
  crowded their neighbours; compact notation has no suffix to shorten
  below a thousand, so drop the fraction digits there.
- Year view shaded alternating bars, because every bucket is its own
  year there — skip the year banding entirely in that view.
- Cap bar width so the three-bar year view doesn't draw slabs, and
  don't repeat the year on both ends of a single-year window label.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: clstaudt

ui/src/components/dashboard/RevenueChart.tsx

@@ -93,9 +93,12 @@ export function RevenueChart() {
   }
 
   const currency = data?.currency || "EUR";
+  // Below a thousand there is no compact suffix to shorten the number, so a
+  // decimal there is just noise: €904, not €903.6.
   const fmtCompact = useCallback(
     (v: number) => new Intl.NumberFormat(undefined, {
-      style: "currency", currency, notation: "compact", maximumFractionDigits: 1,
+      style: "currency", currency, notation: "compact",
+      maximumFractionDigits: Math.abs(v) < 1000 ? 0 : 1,
     }).format(v),
     [currency],
   );
@@ -133,20 +136,24 @@ export function RevenueChart() {
     return indices;
   }, [rows]);
 
+  // Grouping buckets by year is only meaningful when a year spans several of
+  // them — in year view every bucket is its own year.
   const yearBands = useMemo(() => {
+    if (granularity === "year") return [];
     const bands: { year: number; count: number }[] = [];
     for (const row of rows) {
       const last = bands[bands.length - 1];
       if (last && last.year === row.year) last.count += 1;
       else bands.push({ year: row.year, count: 1 });
     }
     return bands;
-  }, [rows]);
+  }, [rows, granularity]);
 
   const windowLabel = useMemo(() => {
     if (!rows.length) return "";
     const first = rows[0], last = rows[rows.length - 1];
     if (granularity === "year") return first.year === last.year ? first.label : `${first.label}–${last.label}`;
+    if (first.year === last.year) return `${first.label} – ${last.label} ${first.year}`;
     return `${first.label} ${first.year} – ${last.label} ${last.year}`;
   }, [rows, granularity]);
 
@@ -254,6 +261,7 @@ export function RevenueChart() {
                     fill={s.color}
                     fillOpacity={s.opacity}
                     radius={isTop ? [3, 3, 0, 0] : [0, 0, 0, 0]}
+                    maxBarSize={72}
                     isAnimationActive={false}
                   >
                     {isTop && (
@@ -276,7 +284,7 @@ export function RevenueChart() {
             <div
               key={`${band.year}-${i}`}
               style={{ flexGrow: band.count }}
-              className={`text-[11px] text-tertiary text-center ${i > 0 ? "border-l border-border" : ""}`}
+              className={`text-[11px] text-secondary text-center ${i > 0 ? "border-l border-border" : ""}`}
             >
               {band.year}
             </div>