From 45906ed6265bbbbc56dd9b1715f6466f462128b4 Mon Sep 17 00:00:00 2001
From: Madhura68 <janpetervisser2@gmail.com>
Date: Sun, 10 May 2026 11:39:52 +0200
Subject: [PATCH] refactor(PBI-77): extract BacklogPageSkeleton from 3
 identical loadings (T-900)

The loading.tsx files for products/[id], sprint/[sprintId], and
sprint/[sprintId]/planning were 100% identical hand-rolled animate-pulse markup.
Extract the shape into components/loading/backlog-page-skeleton.tsx using the
shadcn <Skeleton> component, and re-export it from the three loading files.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 app/(app)/products/[id]/loading.tsx           | 35 +------------------
 .../[id]/sprint/[sprintId]/loading.tsx        | 35 +------------------
 .../sprint/[sprintId]/planning/loading.tsx    | 35 +------------------
 components/loading/backlog-page-skeleton.tsx  | 32 +++++++++++++++++
 4 files changed, 35 insertions(+), 102 deletions(-)
 create mode 100644 components/loading/backlog-page-skeleton.tsx

diff --git a/app/(app)/products/[id]/loading.tsx b/app/(app)/products/[id]/loading.tsx
index 795b2c5..8dcb9c1 100644
--- a/app/(app)/products/[id]/loading.tsx
+++ b/app/(app)/products/[id]/loading.tsx
@@ -1,34 +1 @@
-export default function Loading() {
-  return (
-    <div className="flex flex-col h-full animate-pulse">
-      {/* Header skeleton */}
-      <div className="px-4 py-3 border-b border-border bg-surface-container-low shrink-0 flex items-center justify-between">
-        <div className="space-y-1.5">
-          <div className="h-4 w-32 bg-border rounded" />
-          <div className="h-3 w-48 bg-border/60 rounded" />
-        </div>
-        <div className="h-7 w-24 bg-border rounded" />
-      </div>
-
-      {/* Split pane skeleton */}
-      <div className="flex-1 flex overflow-hidden">
-        {/* Left */}
-        <div className="w-2/5 border-r border-border p-4 space-y-3">
-          <div className="h-4 w-24 bg-border rounded" />
-          {[1, 2, 3, 4, 5].map(i => (
-            <div key={i} className="h-8 bg-border/50 rounded" />
-          ))}
-        </div>
-        {/* Right */}
-        <div className="flex-1 p-4 space-y-3">
-          <div className="h-4 w-16 bg-border rounded" />
-          <div className="flex gap-2 flex-wrap">
-            {[1, 2, 3].map(i => (
-              <div key={i} className="w-28 h-24 bg-border/50 rounded-lg" />
-            ))}
-          </div>
-        </div>
-      </div>
-    </div>
-  )
-}
+export { default } from '@/components/loading/backlog-page-skeleton'
diff --git a/app/(app)/products/[id]/sprint/[sprintId]/loading.tsx b/app/(app)/products/[id]/sprint/[sprintId]/loading.tsx
index 795b2c5..8dcb9c1 100644
--- a/app/(app)/products/[id]/sprint/[sprintId]/loading.tsx
+++ b/app/(app)/products/[id]/sprint/[sprintId]/loading.tsx
@@ -1,34 +1 @@
-export default function Loading() {
-  return (
-    <div className="flex flex-col h-full animate-pulse">
-      {/* Header skeleton */}
-      <div className="px-4 py-3 border-b border-border bg-surface-container-low shrink-0 flex items-center justify-between">
-        <div className="space-y-1.5">
-          <div className="h-4 w-32 bg-border rounded" />
-          <div className="h-3 w-48 bg-border/60 rounded" />
-        </div>
-        <div className="h-7 w-24 bg-border rounded" />
-      </div>
-
-      {/* Split pane skeleton */}
-      <div className="flex-1 flex overflow-hidden">
-        {/* Left */}
-        <div className="w-2/5 border-r border-border p-4 space-y-3">
-          <div className="h-4 w-24 bg-border rounded" />
-          {[1, 2, 3, 4, 5].map(i => (
-            <div key={i} className="h-8 bg-border/50 rounded" />
-          ))}
-        </div>
-        {/* Right */}
-        <div className="flex-1 p-4 space-y-3">
-          <div className="h-4 w-16 bg-border rounded" />
-          <div className="flex gap-2 flex-wrap">
-            {[1, 2, 3].map(i => (
-              <div key={i} className="w-28 h-24 bg-border/50 rounded-lg" />
-            ))}
-          </div>
-        </div>
-      </div>
-    </div>
-  )
-}
+export { default } from '@/components/loading/backlog-page-skeleton'
diff --git a/app/(app)/products/[id]/sprint/[sprintId]/planning/loading.tsx b/app/(app)/products/[id]/sprint/[sprintId]/planning/loading.tsx
index 795b2c5..8dcb9c1 100644
--- a/app/(app)/products/[id]/sprint/[sprintId]/planning/loading.tsx
+++ b/app/(app)/products/[id]/sprint/[sprintId]/planning/loading.tsx
@@ -1,34 +1 @@
-export default function Loading() {
-  return (
-    <div className="flex flex-col h-full animate-pulse">
-      {/* Header skeleton */}
-      <div className="px-4 py-3 border-b border-border bg-surface-container-low shrink-0 flex items-center justify-between">
-        <div className="space-y-1.5">
-          <div className="h-4 w-32 bg-border rounded" />
-          <div className="h-3 w-48 bg-border/60 rounded" />
-        </div>
-        <div className="h-7 w-24 bg-border rounded" />
-      </div>
-
-      {/* Split pane skeleton */}
-      <div className="flex-1 flex overflow-hidden">
-        {/* Left */}
-        <div className="w-2/5 border-r border-border p-4 space-y-3">
-          <div className="h-4 w-24 bg-border rounded" />
-          {[1, 2, 3, 4, 5].map(i => (
-            <div key={i} className="h-8 bg-border/50 rounded" />
-          ))}
-        </div>
-        {/* Right */}
-        <div className="flex-1 p-4 space-y-3">
-          <div className="h-4 w-16 bg-border rounded" />
-          <div className="flex gap-2 flex-wrap">
-            {[1, 2, 3].map(i => (
-              <div key={i} className="w-28 h-24 bg-border/50 rounded-lg" />
-            ))}
-          </div>
-        </div>
-      </div>
-    </div>
-  )
-}
+export { default } from '@/components/loading/backlog-page-skeleton'
diff --git a/components/loading/backlog-page-skeleton.tsx b/components/loading/backlog-page-skeleton.tsx
new file mode 100644
index 0000000..79666b6
--- /dev/null
+++ b/components/loading/backlog-page-skeleton.tsx
@@ -0,0 +1,32 @@
+import { Skeleton } from '@/components/ui/skeleton'
+
+export default function BacklogPageSkeleton() {
+  return (
+    <div className="flex flex-col h-full">
+      <div className="px-4 py-3 border-b border-border bg-surface-container-low shrink-0 flex items-center justify-between">
+        <div className="space-y-1.5">
+          <Skeleton className="h-4 w-32" />
+          <Skeleton className="h-3 w-48" />
+        </div>
+        <Skeleton className="h-7 w-24" />
+      </div>
+
+      <div className="flex-1 flex overflow-hidden">
+        <div className="w-2/5 border-r border-border p-4 space-y-3">
+          <Skeleton className="h-4 w-24" />
+          {[1, 2, 3, 4, 5].map((i) => (
+            <Skeleton key={i} className="h-8 w-full" />
+          ))}
+        </div>
+        <div className="flex-1 p-4 space-y-3">
+          <Skeleton className="h-4 w-16" />
+          <div className="flex gap-2 flex-wrap">
+            {[1, 2, 3].map((i) => (
+              <Skeleton key={i} className="w-28 h-24 rounded-lg" />
+            ))}
+          </div>
+        </div>
+      </div>
+    </div>
+  )
+}