Change event nalme

Author: alexr00

src/github/pullRequestOverview.ts

@@ -48,8 +48,8 @@ export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestMode
 	/**
 	 * Event emitter for when a PR overview becomes active
 	 */
-	private static _onDidChangeActivePullRequest = new vscode.EventEmitter<PullRequestModel>();
-	public static readonly onDidChangeActivePullRequest = PullRequestOverviewPanel._onDidChangeActivePullRequest.event;
+	private static _onVisible = new vscode.EventEmitter<PullRequestModel>();
+	public static readonly onVisible = PullRequestOverviewPanel._onVisible.event;
 
 	private _repositoryDefaultBranch: string;
 	private _existingReviewers: ReviewState[] = [];
@@ -163,10 +163,10 @@ export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestMode
 	protected override onDidChangeViewState(e: vscode.WebviewPanelOnDidChangeViewStateEvent): void {
 		super.onDidChangeViewState(e);
 		this.setVisibilityContext();
-		
+
 		// If the panel becomes visible and we have an item, notify that this PR is active
 		if (this._panel.visible && this._item) {
-			PullRequestOverviewPanel._onDidChangeActivePullRequest.fire(this._item);
+			PullRequestOverviewPanel._onVisible.fire(this._item);
 		}
 	}
 
@@ -320,10 +320,10 @@ export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestMode
 		}
 
 		const result = vscode.window.withProgress({ location: { viewId: 'pr:github' } }, () => this.updateItem(pullRequestModel));
-		
+
 		// Notify that this PR overview is now active
-		PullRequestOverviewPanel._onDidChangeActivePullRequest.fire(pullRequestModel);
-		
+		PullRequestOverviewPanel._onVisible.fire(pullRequestModel);
+
 		return result;
 	}
 
@@ -824,7 +824,7 @@ export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestMode
 	 * Static dispose method to clean up static resources
 	 */
 	public static dispose() {
-		PullRequestOverviewPanel._onDidChangeActivePullRequest.dispose();
+		PullRequestOverviewPanel._onVisible.dispose();
 	}
 }
 

src/test/github/pullRequestOverview.test.ts

@@ -155,7 +155,7 @@ describe('PullRequestOverview', function () {
 			// Listen for the event
 			let eventFired = false;
 			let eventPR: PullRequestModel | undefined;
-			const disposable = PullRequestOverviewPanel.onDidChangeActivePullRequest(pr => {
+			const disposable = PullRequestOverviewPanel.onVisible(pr => {
 				eventFired = true;
 				eventPR = pr;
 			});
@@ -193,7 +193,7 @@ describe('PullRequestOverview', function () {
 			// Listen for the event
 			let eventCount = 0;
 			let lastEventPR: PullRequestModel | undefined;
-			const disposable = PullRequestOverviewPanel.onDidChangeActivePullRequest(pr => {
+			const disposable = PullRequestOverviewPanel.onVisible(pr => {
 				eventCount++;
 				lastEventPR = pr;
 			});

src/view/prsTreeDataProvider.ts

@@ -109,7 +109,7 @@ export class PullRequestsTreeDataProvider extends Disposable implements vscode.T
 		this._register(this._copilotManager.onDidCreatePullRequest(() => this.refresh()));
 
 		// Listen for PR overview panel changes to sync the tree view
-		this._register(PullRequestOverviewPanel.onDidChangeActivePullRequest(pullRequest => {
+		this._register(PullRequestOverviewPanel.onVisible(pullRequest => {
 			this.syncWithActivePullRequest(pullRequest);
 		}));
 
@@ -187,7 +187,7 @@ export class PullRequestsTreeDataProvider extends Disposable implements vscode.T
 		try {
 			// First expand the pull request in the tree to make sure it's visible
 			await this.expandPullRequest(pullRequest);
-			
+
 			// Find the PR node in the tree and reveal it
 			const prNode = await this.findPRNode(pullRequest);
 			if (prNode) {
@@ -213,7 +213,7 @@ export class PullRequestsTreeDataProvider extends Disposable implements vscode.T
 				if (found) return found;
 			} else if (child instanceof CategoryTreeNode) {
 				const found = await this.findPRNodeInCategory(child, pullRequest);
-				if (found) return found;  
+				if (found) return found;
 			}
 		}
 		return undefined;