Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create Milestone #3686

Merged
merged 5 commits into from
Jul 7, 2022
Merged

Create Milestone #3686

merged 5 commits into from
Jul 7, 2022

Conversation

benibenj
Copy link
Contributor

@benibenj benibenj commented Jul 5, 2022

When clicking on add milestone, a dropdown appears with all milestones. At the top right there is a plus which will create an inputbox where you can give the name of the new milestone. If you have already entered it and click on the plus (add), the inputbox appears with the text prefilled.

@benibenj benibenj requested a review from alexr00 July 5, 2022 09:29
Copy link
Member

@alexr00 alexr00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just a few questions.

src/github/pullRequestOverview.ts Outdated Show resolved Hide resolved

const inputBox = vscode.window.createInputBox();
inputBox.title = 'Create new milestone';
inputBox.placeholder = 'New milestone name';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I find the quick pick with the button/title bar without any title text to be a bit bare looking. Can you move the placeholder to the title?

image

Copy link
Member

@alexr00 alexr00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

});

quickPick.show();
quickPick.items = await getMilestoneOptions(this._folderRepositoryManager);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just curious: why did you remove the quickPick.busy setting?

@benibenj benibenj merged commit 28fdd8f into main Jul 7, 2022
@benibenj benibenj deleted the benibenj/createMilestone branch July 7, 2022 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants