feat: implement inline editing for comments

Author: CREDO23

surfsense_web/components/chat-comments/comment-composer/comment-composer.tsx

@@ -86,8 +86,9 @@ export function CommentComposer({
 	onSubmit,
 	onCancel,
 	autoFocus = false,
+	initialValue = "",
 }: CommentComposerProps) {
-	const [displayContent, setDisplayContent] = useState("");
+	const [displayContent, setDisplayContent] = useState(initialValue);
 	const [insertedMentions, setInsertedMentions] = useState<InsertedMention[]>([]);
 	const [mentionState, setMentionState] = useState<MentionState>({
 		isActive: false,

surfsense_web/components/chat-comments/comment-composer/types.ts

@@ -9,6 +9,7 @@ export interface CommentComposerProps {
 	onSubmit: (content: string) => void;
 	onCancel?: () => void;
 	autoFocus?: boolean;
+	initialValue?: string;
 }
 
 export interface MentionState {

surfsense_web/components/chat-comments/comment-item/comment-item.tsx

@@ -6,6 +6,7 @@ import { currentUserAtom } from "@/atoms/user/user-query.atoms";
 import { Avatar, AvatarFallback, AvatarImage } from "@/components/ui/avatar";
 import { Button } from "@/components/ui/button";
 import { cn } from "@/lib/utils";
+import { CommentComposer } from "../comment-composer/comment-composer";
 import { CommentActions } from "./comment-actions";
 import type { CommentItemProps } from "./types";
 
@@ -99,9 +100,14 @@ function renderMentions(content: string): React.ReactNode {
 export function CommentItem({
 	comment,
 	onEdit,
+	onEditSubmit,
+	onEditCancel,
 	onDelete,
 	onReply,
 	isReply = false,
+	isEditing = false,
+	members = [],
+	membersLoading = false,
 }: CommentItemProps) {
 	const [{ data: currentUser }] = useAtom(currentUserAtom);
 
@@ -111,6 +117,10 @@ export function CommentItem({
 		: comment.author?.displayName || comment.author?.email.split("@")[0] || "Unknown";
 	const email = comment.author?.email || "";
 
+	const handleEditSubmit = (content: string) => {
+		onEditSubmit?.(comment.id, content);
+	};
+
 	return (
 		<div className={cn("group flex gap-3")}>
 			<Avatar className="size-8 shrink-0">
@@ -131,21 +141,38 @@ export function CommentItem({
 					{comment.isEdited && (
 						<span className="shrink-0 text-xs text-muted-foreground">(edited)</span>
 					)}
-					<div className="ml-auto">
-						<CommentActions
-							canEdit={comment.canEdit}
-							canDelete={comment.canDelete}
-							onEdit={() => onEdit?.(comment.id)}
-							onDelete={() => onDelete?.(comment.id)}
-						/>
-					</div>
+					{!isEditing && (
+						<div className="ml-auto">
+							<CommentActions
+								canEdit={comment.canEdit}
+								canDelete={comment.canDelete}
+								onEdit={() => onEdit?.(comment.id)}
+								onDelete={() => onDelete?.(comment.id)}
+							/>
+						</div>
+					)}
 				</div>
 
-				<div className="mt-1 text-sm text-foreground whitespace-pre-wrap wrap-break-word">
-					{renderMentions(comment.contentRendered)}
-				</div>
+				{isEditing ? (
+					<div className="mt-1">
+						<CommentComposer
+							members={members}
+							membersLoading={membersLoading}
+							placeholder="Edit your comment..."
+							submitLabel="Save"
+							onSubmit={handleEditSubmit}
+							onCancel={onEditCancel}
+							initialValue={comment.content}
+							autoFocus
+						/>
+					</div>
+				) : (
+					<div className="mt-1 text-sm text-foreground whitespace-pre-wrap wrap-break-word">
+						{renderMentions(comment.contentRendered)}
+					</div>
+				)}
 
-				{!isReply && onReply && (
+				{!isReply && onReply && !isEditing && (
 					<Button
 						variant="ghost"
 						size="sm"

surfsense_web/components/chat-comments/comment-item/types.ts

@@ -20,9 +20,14 @@ export interface CommentData {
 export interface CommentItemProps {
 	comment: CommentData;
 	onEdit?: (commentId: number) => void;
+	onEditSubmit?: (commentId: number, content: string) => void;
+	onEditCancel?: () => void;
 	onDelete?: (commentId: number) => void;
 	onReply?: (commentId: number) => void;
 	isReply?: boolean;
+	isEditing?: boolean;
+	members?: Array<{ id: string; displayName: string | null; email: string; avatarUrl?: string | null }>;
+	membersLoading?: boolean;
 }
 
 export interface CommentActionsProps {

surfsense_web/components/chat-comments/comment-thread/comment-thread.tsx

@@ -18,6 +18,7 @@ export function CommentThread({
 }: CommentThreadProps) {
 	const [isRepliesExpanded, setIsRepliesExpanded] = useState(true);
 	const [isReplyComposerOpen, setIsReplyComposerOpen] = useState(false);
+	const [editingCommentId, setEditingCommentId] = useState<number | null>(null);
 
 	const parentComment = {
 		id: thread.id,
@@ -45,6 +46,19 @@ export function CommentThread({
 		setIsReplyComposerOpen(false);
 	};
 
+	const handleEditStart = (commentId: number) => {
+		setEditingCommentId(commentId);
+	};
+
+	const handleEditSubmit = (commentId: number, content: string) => {
+		onEditComment(commentId, content);
+		setEditingCommentId(null);
+	};
+
+	const handleEditCancel = () => {
+		setEditingCommentId(null);
+	};
+
 	const hasReplies = thread.replies.length > 0;
 	const showReplies = thread.replies.length === 1 || isRepliesExpanded;
 
@@ -53,8 +67,13 @@ export function CommentThread({
 			{/* Parent comment */}
 			<CommentItem
 				comment={parentComment}
-				onEdit={(id) => onEditComment(id, "")}
+				onEdit={handleEditStart}
+				onEditSubmit={handleEditSubmit}
+				onEditCancel={handleEditCancel}
 				onDelete={onDeleteComment}
+				isEditing={editingCommentId === parentComment.id}
+				members={members}
+				membersLoading={membersLoading}
 			/>
 
 			{/* Replies and actions - using flex layout with connector */}
@@ -92,8 +111,13 @@ export function CommentThread({
 										key={reply.id}
 										comment={reply}
 										isReply
-										onEdit={(id) => onEditComment(id, "")}
+										onEdit={handleEditStart}
+										onEditSubmit={handleEditSubmit}
+										onEditCancel={handleEditCancel}
 										onDelete={onDeleteComment}
+										isEditing={editingCommentId === reply.id}
+										members={members}
+										membersLoading={membersLoading}
 									/>
 								))}
 							</div>