From bbd4cba1c162e36b0b7cd049cbdf232c75f3c132 Mon Sep 17 00:00:00 2001
From: highperfocused <highperfocused@pm.me>
Date: Sat, 22 Nov 2025 21:17:49 +0100
Subject: [PATCH] Refactor CommentsSection to simplify error handling and
 improve layout structure

---
 src/components/comments/CommentsSection.tsx | 118 ++++++++++----------
 1 file changed, 56 insertions(+), 62 deletions(-)

diff --git a/src/components/comments/CommentsSection.tsx b/src/components/comments/CommentsSection.tsx
index a269934..6de0e99 100644
--- a/src/components/comments/CommentsSection.tsx
+++ b/src/components/comments/CommentsSection.tsx
@@ -29,72 +29,66 @@ export function CommentsSection({
 
   if (error) {
     return (
-      <Card className="rounded-none sm:rounded-lg mx-0 sm:mx-0">
-        <CardContent className="px-2 py-6 sm:p-6">
-          <div className="text-center text-muted-foreground">
-            <MessageSquare className="h-8 w-8 mx-auto mb-2 opacity-50" />
-            <p>Failed to load comments</p>
-          </div>
-        </CardContent>
-      </Card>
+      <div className="text-center text-muted-foreground py-6">
+        <MessageSquare className="h-8 w-8 mx-auto mb-2 opacity-50" />
+        <p>Failed to load comments</p>
+      </div>
     );
   }
 
   return (
-    <Card className={cn("rounded-none sm:rounded-lg mx-0 sm:mx-0", className)}>
-      <CardHeader className="px-2 pt-6 pb-4 sm:p-6">
-        <CardTitle className="flex items-center space-x-2">
-          <MessageSquare className="h-5 w-5" />
-          <span>{title}</span>
-          {!isLoading && (
-            <span className="text-sm font-normal text-muted-foreground">
-              ({comments.length})
-            </span>
-          )}
-        </CardTitle>
-      </CardHeader>
-      <CardContent className="px-2 pb-6 pt-4 sm:p-6 sm:pt-0 space-y-6">
-        {/* Comment Form */}
-        <CommentForm root={root} />
-
-        {/* Comments List */}
-        {isLoading ? (
-          <div className="space-y-4">
-            {[...Array(3)].map((_, i) => (
-              <Card key={i} className="bg-card/50">
-                <CardContent className="p-4">
-                  <div className="space-y-3">
-                    <div className="flex items-center space-x-3">
-                      <Skeleton className="h-8 w-8 rounded-full" />
-                      <div className="space-y-1">
-                        <Skeleton className="h-4 w-24" />
-                        <Skeleton className="h-3 w-16" />
-                      </div>
-                    </div>
-                    <Skeleton className="h-16 w-full" />
-                  </div>
-                </CardContent>
-              </Card>
-            ))}
-          </div>
-        ) : comments.length === 0 ? (
-          <div className="text-center py-8 text-muted-foreground">
-            <MessageSquare className="h-12 w-12 mx-auto mb-4 opacity-30" />
-            <p className="text-lg font-medium mb-2">{emptyStateMessage}</p>
-            <p className="text-sm">{emptyStateSubtitle}</p>
-          </div>
-        ) : (
-          <div className="space-y-4">
-            {comments.map((comment) => (
-              <Comment
-                key={comment.id}
-                root={root}
-                comment={comment}
-              />
-            ))}
-          </div>
+    <div className={cn("space-y-6", className)}>
+      {/* Header */}
+      <div className="flex items-center space-x-2">
+        <MessageSquare className="h-5 w-5" />
+        <h2 className="text-xl font-semibold">{title}</h2>
+        {!isLoading && (
+          <span className="text-sm font-normal text-muted-foreground">
+            ({comments.length})
+          </span>
         )}
-      </CardContent>
-    </Card>
+      </div>
+
+      {/* Comment Form */}
+      <CommentForm root={root} />
+
+      {/* Comments List */}
+      {isLoading ? (
+        <div className="space-y-4">
+          {[...Array(3)].map((_, i) => (
+            <Card key={i} className="bg-card/50">
+              <CardContent className="p-4">
+                <div className="space-y-3">
+                  <div className="flex items-center space-x-3">
+                    <Skeleton className="h-8 w-8 rounded-full" />
+                    <div className="space-y-1">
+                      <Skeleton className="h-4 w-24" />
+                      <Skeleton className="h-3 w-16" />
+                    </div>
+                  </div>
+                  <Skeleton className="h-16 w-full" />
+                </div>
+              </CardContent>
+            </Card>
+          ))}
+        </div>
+      ) : comments.length === 0 ? (
+        <div className="text-center py-8 text-muted-foreground">
+          <MessageSquare className="h-12 w-12 mx-auto mb-4 opacity-30" />
+          <p className="text-lg font-medium mb-2">{emptyStateMessage}</p>
+          <p className="text-sm">{emptyStateSubtitle}</p>
+        </div>
+      ) : (
+        <div className="space-y-4">
+          {comments.map((comment) => (
+            <Comment
+              key={comment.id}
+              root={root}
+              comment={comment}
+            />
+          ))}
+        </div>
+      )}
+    </div>
   );
 }
\ No newline at end of file