Fixes #708 - Prompt for file path when missing from revision by mogelbrod · Pull Request #2825 · gitkraken/vscode-gitlens

mogelbrod · 2023-07-20T18:10:36Z

Description

Fixes #708 - Prompt for file path when missing from revision

Improves the openFileAtRevision() function by:

Asking user to enter another path when the requested file doesn't exist in the
selected revision.
Showing an error message on subsequent failure (or if an error is thrown).

The above function is used by a number of commands:

gitlens.openFileRevision
gitlens.openFileRevisionFrom
gitlens.openRevisionFile

Checklist

I have followed the guidelines in the Contributing document
My changes follow the coding style of this project
My changes build without any errors or warnings
My changes have been formatted and linted
My changes include any required corresponding changes to the documentation (including CHANGELOG.md and README.md)
My changes have been rebased and squashed to the minimal number (typically 1) of relevant commits
My changes have a descriptive commit message with a short title, including a Fixes $XXX - or Closes #XXX - prefix to auto-close the issue that your PR addresses

mogelbrod · 2023-08-29T14:31:33Z

Would love to get this reviewed (and hopefully merged) @eamodio 😃

mogelbrod · 2023-11-16T19:00:43Z

Rebased the PR onto the latest main branch again, eagerly awaiting a review 😄

eamodio · 2023-11-27T06:30:26Z

@mogelbrod Thanks for this -- I've started taking a pass through this and having issues with the OpenFileByRevisionCommand -- it needs an active editor (though shouldn't really) and the keyboarding doesn't feel right. In GitLens (and VS Code in general), the using the right arrow key in a quick pick, when appropriate, leaves the quick pick open and acts upon the selected item (opening it, etc). This functionality should be moved into the pickFileAtRevision quick pick rather than in the refs picker.

But before I dig more into this, I'd love to have the 2 parts split apart. So split out the OpenFileByRevisionCommand into a separate PR, as I think there are more UX implications (naming, etc).

eamodio · 2023-11-27T06:34:22Z

src/git/actions/commit.ts

+export async function pickFileAtRevision(
+	uri: GitUri,
+	options: {
+		title: string;
+		initialPath?: string;
+	},
+): Promise<Uri | undefined> {
+	const disposables: Disposable[] = [];
+	try {
+		const picker = window.createQuickPick();
+		Object.assign(picker, {
+			title: options.title,
+			value: options.initialPath ?? uri.relativePath,
+			placeholder: 'Enter path to file...',
+			busy: true,
+			ignoreFocusOut: getQuickPickIgnoreFocusOut(),
+		});
+		picker.show();
+
+		const tree = await Container.instance.git.getTreeForRevision(uri.repoPath, uri.sha!);
+		picker.items = tree.filter(file => file.type === 'blob').map((file): QuickPickItem => ({ label: file.path }));
+		picker.busy = false;
+
+		const pick = await new Promise<string | undefined>(resolve => {
+			disposables.push(
+				picker,
+				picker.onDidHide(() => resolve(undefined)),
+				picker.onDidAccept(() => {
+					if (picker.activeItems.length === 0) return;
+					resolve(picker.activeItems[0].label);
+				}),
+			);
+		});
+
+		return pick
+			? Container.instance.git.getRevisionUri(uri.sha!, `${uri.repoPath}/${pick}`, uri.repoPath!)
+			: undefined;
+	} finally {
+		disposables.forEach(d => {
+			d.dispose();
+		});


Let's move this into src/quickpicks/revisionPicker.ts similar to the other pickers

Sure! Should it be called fileRevisionPicker (or a variation thereof) or revisionPicker? I'm leaning towards the former since it's technically picking a file at a specific revision, rather than picking a revision (unless the "file" part is implicit for revisions?).

src/git/actions/commit.ts

eamodio · 2023-11-27T06:38:09Z

src/git/actions/commit.ts

+		const tree = await Container.instance.git.getTreeForRevision(uri.repoPath, uri.sha!);
+		picker.items = tree.filter(file => file.type === 'blob').map((file): QuickPickItem => ({ label: file.path }));


I'd like to try only putting the base filename as the label and putting the rest of the folder path in the description -- closer to the vscode picker. Will need to enable picker.matchOnDescription as well.

I tried this now, but vscode appears to not match any of the following items when
picker.value = 'directory/file.tsx':

[ { label: 'file.tsx', description: 'directory' }, { label: 'file.tsx', description: 'directory/' }, }

eamodio · 2023-11-27T06:39:03Z

src/git/actions/commit.ts

+		const pick = await new Promise<string | undefined>(resolve => {
+			disposables.push(
+				picker,
+				picker.onDidHide(() => resolve(undefined)),
+				picker.onDidAccept(() => {
+					if (picker.activeItems.length === 0) return;
+					resolve(picker.activeItems[0].label);
+				}),
+			);
+		});


It would be nice to add the extra keyboard (right arrow) support here to open the selected file in a preview editor.

I didn't know about this functionality 😄 Not sure how though, do you have any pointers to existing implementations?

Would it potentially break text selections using left/right arrow keys in the text field?

mogelbrod · 2023-11-29T17:21:56Z

src/quickpicks/revisionPicker.ts

+			// FIXME: Remove this unless we opt to show the directory in the description
+			// const parsed = path.parse(file.path)
+			// return { label: parsed.base, description: parsed.dir }


Left this code in if you wanted to try it out yourself @eamodio. It does depend on path, which appears to be a forbidden import.

mogelbrod · 2023-11-29T17:28:27Z

Thanks for taking the time to dig into this @eamodio!

I've extracted the openFileByRevision command addition into a separate PR. It's currently marked as draft - I'll update it to reflect all changes once this PR lands in main.

- Avoids breaking ctrl+[left|right] in input - Introduces structured `keyboard` object to handle keypress event bindings and actions

Improves the `openFileAtRevision()` function by: - Asking user to enter another path when the requested file doesn't exist in the selected revision. - Showing an error message on subsequent failure (or if an error is thrown). The above function is used by a number of commands: - `gitlens.openFileRevision` - `gitlens.openFileRevisionFrom` - `gitlens.openRevisionFile`

Adds keyboard nav to revision picker

eamodio

Thanks for splitting that up -- looks great. I've cleaned up the UX and added some keyboard support.

Thank you so much for your contribution!

mogelbrod force-pushed the main branch 2 times, most recently from f27c94d to 9ea07d3 Compare July 20, 2023 18:12

mogelbrod marked this pull request as ready for review July 20, 2023 18:34

mogelbrod force-pushed the main branch from 9ea07d3 to 8cb86b8 Compare July 23, 2023 13:41

mogelbrod changed the title ~~Fixes #708 - Prompt for file path when missing from revision~~ Adds openFileByRevision command & prompts for file path when missing from revision Jul 23, 2023

mogelbrod force-pushed the main branch from dcada29 to 249c93a Compare November 16, 2023 18:53

mogelbrod changed the title ~~Adds openFileByRevision command & prompts for file path when missing from revision~~ Add ability to open any file on any branch using quickpick Nov 16, 2023

mogelbrod mentioned this pull request Nov 16, 2023

open any file from another branch #708

Closed

eamodio requested changes Nov 27, 2023

View reviewed changes

mogelbrod force-pushed the main branch from 249c93a to 8a9a745 Compare November 29, 2023 17:19

mogelbrod changed the title ~~Add ability to open any file on any branch using quickpick~~ Fixes #708 - Prompt for file path when missing from revision Nov 29, 2023

mogelbrod commented Nov 29, 2023

View reviewed changes

mogelbrod requested a review from eamodio November 29, 2023 17:22

mogelbrod mentioned this pull request Nov 29, 2023

Fixes #2515 - Adds openFileByRevision command #3036

Open

7 tasks

eamodio added 5 commits December 6, 2023 01:18

Fixes oid vs ref on trees

aab2b74

Adds tuple labels

6935b4c

Improves quickpick keyboard navigation

7981bd5

- Avoids breaking ctrl+[left|right] in input - Introduces structured `keyboard` object to handle keypress event bindings and actions

Improves the UX of the revision picker

2fa7fe2

Adds keyboard nav to revision picker

eamodio force-pushed the main branch from 8a9a745 to 2fa7fe2 Compare December 6, 2023 07:01

eamodio approved these changes Dec 6, 2023

View reviewed changes

eamodio merged commit faaf78f into gitkraken:main Dec 6, 2023

eamodio added this to the 14.6 milestone Dec 6, 2023

		const tree = await Container.instance.git.getTreeForRevision(uri.repoPath, uri.sha!);
		picker.items = tree.filter(file => file.type === 'blob').map((file): QuickPickItem => ({ label: file.path }));

Conversation

mogelbrod commented Jul 20, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Checklist

Uh oh!

mogelbrod commented Aug 29, 2023

Uh oh!

mogelbrod commented Nov 16, 2023

Uh oh!

eamodio commented Nov 27, 2023

Uh oh!

eamodio Nov 27, 2023

Choose a reason for hiding this comment

Uh oh!

mogelbrod Nov 29, 2023

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

eamodio Nov 27, 2023

Choose a reason for hiding this comment

Uh oh!

mogelbrod Nov 29, 2023

Choose a reason for hiding this comment

Uh oh!

eamodio Nov 27, 2023

Choose a reason for hiding this comment

Uh oh!

mogelbrod Nov 29, 2023

Choose a reason for hiding this comment

Uh oh!

mogelbrod Nov 29, 2023

Choose a reason for hiding this comment

Uh oh!

mogelbrod commented Nov 29, 2023

Uh oh!

eamodio left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

mogelbrod commented Jul 20, 2023 •

edited

Loading