Splits commands in home for viewing pull request changes

Author: d13

src/webviews/apps/plus/home/components/active-work.ts

@@ -219,7 +219,7 @@ export class GlActiveWork extends SignalWatcher(LitElement) {
 				html`<action-item
 					label="Open Pull Request Changes"
 					icon="request-changes"
-					href=${createCommandLink('gitlens.home.openPullRequestComparison', branchRefs)}
+					href=${createCommandLink('gitlens.home.openPullRequestChanges', branchRefs)}
 				></action-item>`,
 			);
 			actions.push(

src/webviews/apps/plus/home/components/branch-section.ts

@@ -117,9 +117,6 @@ export const headingLoaderStyles = css`
 `;
 
 export const branchCardStyles = css`
-	:host {
-		--gl-card-hover-background: color-mix(in lab, var(--vscode-sideBar-background) 100%, #fff 8%);
-	}
 	.branch-item {
 		position: relative;
 	}
@@ -196,13 +193,9 @@ export const branchCardStyles = css`
 		right: 0.4rem;
 		bottom: 0.4rem;
 		padding: 0.2rem 0.4rem;
-		background-color: var(--gl-card-background);
-	}
-
-	.branch-item:hover .branch-item__actions,
-	.branch-item:focus-within .branch-item__actions {
 		background-color: var(--gl-card-hover-background);
 	}
+
 	.branch-item:not(:focus-within):not(:hover) .branch-item__actions {
 		${srOnlyStyles}
 	}
@@ -358,7 +351,7 @@ export class GlBranchCard extends LitElement {
 				html`<action-item
 					label="Open Pull Request Changes"
 					icon="request-changes"
-					href=${this.createCommandLink('gitlens.home.openPullRequestComparison')}
+					href=${this.createCommandLink('gitlens.home.openPullRequestChanges')}
 				></action-item>`,
 			);
 			actions.push(

src/webviews/home/homeWebview.ts

@@ -9,6 +9,7 @@ import type { ContextKeys } from '../../constants.context';
 import type { HomeTelemetryContext } from '../../constants.telemetry';
 import type { Container } from '../../container';
 import { executeGitCommand } from '../../git/actions';
+import { openComparisonChanges } from '../../git/actions/commit';
 import * as RepoActions from '../../git/actions/repository';
 import type { BranchContributorOverview } from '../../git/gitProvider';
 import type { GitBranch } from '../../git/models/branch';
@@ -266,6 +267,7 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 				() => this.container.subscription.validate({ force: true }),
 				this,
 			),
+			registerCommand('gitlens.home.openPullRequestChanges', this.pullRequestChanges, this),
 			registerCommand('gitlens.home.openPullRequestComparison', this.pullRequestCompare, this),
 			registerCommand('gitlens.home.openPullRequestOnRemote', this.pullRequestViewOnRemote, this),
 			registerCommand('gitlens.home.createPullRequest', this.pullRequestCreate, this),
@@ -773,11 +775,27 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 		if (pr?.refs?.base == null || pr.refs.head == null) return;
 
 		const comparisonRefs = getComparisonRefsForPullRequest(refs.repoPath, pr.refs);
-		return this.container.views.searchAndCompare
-			.compare(comparisonRefs.repoPath, comparisonRefs.head, comparisonRefs.base)
-			.then(compareNode => {
-				executeCommand(Commands.ViewsOpenAllChangedFileDiffs, compareNode);
-			});
+		return this.container.views.searchAndCompare.compare(
+			comparisonRefs.repoPath,
+			comparisonRefs.head,
+			comparisonRefs.base,
+		);
+	}
+
+	private async pullRequestChanges(refs: BranchRef) {
+		const pr = await this.findPullRequest(refs);
+		if (pr?.refs?.base == null || pr.refs.head == null) return;
+
+		const comparisonRefs = getComparisonRefsForPullRequest(refs.repoPath, pr.refs);
+		return openComparisonChanges(
+			this.container,
+			{
+				repoPath: comparisonRefs.repoPath,
+				lhs: comparisonRefs.base.ref,
+				rhs: comparisonRefs.head.ref,
+			},
+			{ title: `Changes in Pull Request #${pr.id}` },
+		);
 	}
 
 	private async pullRequestViewOnRemote(refs: BranchRef, clipboard?: boolean) {