diff --git a/frontend/src/components/dialogs/tasks/GitHubCommentsDialog.tsx b/frontend/src/components/dialogs/tasks/GitHubCommentsDialog.tsx index 9863b62f..567842f7 100644 --- a/frontend/src/components/dialogs/tasks/GitHubCommentsDialog.tsx +++ b/frontend/src/components/dialogs/tasks/GitHubCommentsDialog.tsx @@ -87,9 +87,13 @@ const GitHubCommentsDialogImpl = NiceModal.create( const errorMessage = isError ? getErrorMessage(error) : null; return ( - + { if (e.key === 'Escape') { e.stopPropagation(); @@ -98,92 +102,97 @@ const GitHubCommentsDialogImpl = NiceModal.create( } }} > - + {t('tasks:githubComments.dialog.title')} -
- {errorMessage ? ( - - - {errorMessage} - - ) : isLoading ? ( -
- -
- ) : comments.length === 0 ? ( -

- {t('tasks:githubComments.dialog.noComments')} -

- ) : ( - <> -
- - {t('tasks:githubComments.dialog.selectedCount', { - selected: selectedIds.size, - total: comments.length, +
+
+ {errorMessage ? ( + + + {errorMessage} + + ) : isLoading ? ( +
+ +
+ ) : comments.length === 0 ? ( +

+ {t('tasks:githubComments.dialog.noComments')} +

+ ) : ( + <> +
+ + {t('tasks:githubComments.dialog.selectedCount', { + selected: selectedIds.size, + total: comments.length, + })} + + +
+
+ {comments.map((comment) => { + const id = getCommentId(comment); + return ( +
+ toggleSelection(id)} + className="mt-3" + /> + toggleSelection(id)} + className="flex-1 min-w-0" + /> +
+ ); })} - - -
-
- {comments.map((comment) => { - const id = getCommentId(comment); - return ( -
- toggleSelection(id)} - className="mt-3" - /> - toggleSelection(id)} - className="block w-full max-w-none flex-1" - /> -
- ); - })} -
- - )} +
+ + )} +
{!errorMessage && !isLoading && comments.length > 0 && ( - + diff --git a/frontend/src/components/ui/dialog.tsx b/frontend/src/components/ui/dialog.tsx index eacfc2ef..6cae8b8b 100644 --- a/frontend/src/components/ui/dialog.tsx +++ b/frontend/src/components/ui/dialog.tsx @@ -116,7 +116,7 @@ const Dialog = React.forwardRef<
void; onDoubleClick?: (e: React.MouseEvent) => void; className?: string; @@ -111,6 +111,7 @@ function FullCard({ line, diffHunk, onClick, + variant, className, }: GitHubCommentCardProps) { const { t } = useTranslation('tasks'); @@ -120,7 +121,8 @@ function FullCard({ return (
; } - + // Both 'full' and 'list' use FullCard, just with different styling return ; }