diff --git a/src/components/Revision.svelte b/src/components/Revision.svelte index f19c61a97..fd72788dc 100644 --- a/src/components/Revision.svelte +++ b/src/components/Revision.svelte @@ -58,7 +58,9 @@ let focusReply: boolean = $state(false); let hideChanges = $state(false); - let hideDiscussion = $state(false); + let hideDiscussion = $state( + revision.discussion === undefined || revision.discussion.length === 0, + ); let hideReviews = $state(false); let topLevelReplyOpen = $state(false); let filesExpanded = $state(true); @@ -88,7 +90,10 @@ patchId; hideReviews = false; - hideDiscussion = false; + hideDiscussion = + revision.discussion === undefined || revision.discussion.length === 0; + focusReply = false; + topLevelReplyOpen = false; hideChanges = false; }); @@ -253,6 +258,7 @@ behavior: "smooth", block: "center", }); + await tick(); focusReply = true; } @@ -350,9 +356,6 @@ repoDelegates.map(delegate => delegate.did), revision.author.did, ) && partial(editRevision, revision.id)}> - {#snippet actions()} - - {/snippet} @@ -411,16 +414,41 @@
-
+
(hideDiscussion = !hideDiscussion)}> -
Discussion
+
+ Discussion + {revision.discussion?.length ?? 0} + +
+
+ { + if (hideDiscussion) { + hideDiscussion = false; + } else { + if ( + revision.discussion === undefined || + revision.discussion.length === 0 + ) { + hideDiscussion = true; + } + } + await toggleReply(); + }}> + + Comment + +
-
+
{#each threads as thread} (topLevelReplyOpen = false) + ? () => { + if ( + revision.discussion === undefined || + revision.discussion.length === 0 + ) { + hideDiscussion = !hideDiscussion; + } + topLevelReplyOpen = false; + } : undefined} placeholder="Leave a comment" submit={partial(createComment)} /> @@ -454,10 +490,7 @@
- (hideChanges = !hideChanges)}> + (hideChanges = !hideChanges)}>
Changes