Merge pull request #10406 from gitbutlerapp/copilot/fix-6bea0204-7557-4d3a-a1a5-92fe21fe0869

Fix CommitTitle to show context-appropriate messages for empty commits

Author: estib-vega

apps/desktop/src/components/BranchCommitList.svelte

@@ -272,6 +272,7 @@
 								{active}
 								onclick={() => handleCommitClick(commit.id, true)}
 								disableCommitActions={false}
+								editable={!!stackId}
 							/>
 						{/if}
 					{/each}
@@ -414,6 +415,7 @@
 								{active}
 								onclick={() => handleCommitClick(commit.id, false)}
 								disableCommitActions={false}
+								editable={!!stackId}
 							>
 								{#snippet menu({ rightClickTrigger })}
 									{@const data = {

apps/desktop/src/components/CommitRow.svelte

@@ -27,6 +27,7 @@
 		active?: boolean;
 		hasConflicts?: boolean;
 		disabled?: boolean;
+		editable?: boolean;
 		menu?: Snippet<[{ rightClickTrigger: HTMLElement }]>;
 		onclick?: () => void;
 	};
@@ -74,6 +75,7 @@
 		disabled,
 		hasConflicts,
 		active,
+		editable,
 		onclick,
 		menu,
 		...args
@@ -146,7 +148,7 @@
 		{/if}
 
 		<div class="commit-name truncate">
-			<CommitTitle {commitMessage} truncate className="text-13 text-semibold" />
+			<CommitTitle {commitMessage} truncate className="text-13 text-semibold" {editable} />
 		</div>
 
 		{#if !args.disableCommitActions}

apps/desktop/src/components/CommitTitle.svelte

@@ -6,17 +6,18 @@
 		truncate?: boolean;
 		commitMessage: string;
 		className?: string;
+		editable?: boolean;
 	};
 
-	const { commitMessage, truncate, className }: Props = $props();
+	const { commitMessage, truncate, className, editable }: Props = $props();
 
 	const title = $derived(splitMessage(commitMessage).title);
 
 	function getTitle() {
 		if (title) {
 			return title;
 		}
-		return 'Empty commit. Drag changes here';
+		return editable ? 'Empty commit. Drag changes here' : 'Empty commit';
 	}
 </script>
 

apps/desktop/src/components/CommitView.svelte

@@ -166,6 +166,7 @@
 					truncate
 					commitMessage={commit.message}
 					className="text-14 text-semibold text-body"
+					editable={!isReadOnly}
 				/>
 			{/snippet}