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

Tasks: Configure Task opens menu with one item #40508

Closed
Gruntfuggly opened this issue Dec 19, 2017 · 9 comments
Closed

Tasks: Configure Task opens menu with one item #40508

Gruntfuggly opened this issue Dec 19, 2017 · 9 comments
Assignees
Labels
feature-request Request for new features or functionality tasks Task system issues verified Verification succeeded
Milestone

Comments

@Gruntfuggly
Copy link

I only have one tasks file with one task, so when I do Tasks: Configure Task, why does it pop up a menu with one option that I have to select?

With version 0.1.0, the only option is to open the tasks file (regardless of the number of defined tasks). With 2.0.0 it lists the defined tasks, but if I only have one, I shouldn't need to select it from the list.

It's not exactly high priority, or even particularly frustrating, but it seems wrong.

Maybe if I type in a new task name it should create an empty task in the config file, with the correct format?

@vscodebot vscodebot bot added the tasks Task system issues label Dec 19, 2017
@dbaeumer dbaeumer added this to the Backlog milestone Dec 19, 2017
@dbaeumer dbaeumer added the feature-request Request for new features or functionality label Dec 19, 2017
@dbaeumer
Copy link
Member

For the configure it would make sense. To terminate or start we wanted to keep it consistent so that users see what will happen.

@alexr00
Copy link
Member

alexr00 commented Oct 3, 2019

Should also include "Show Output".

@alexr00 alexr00 assigned alexr00 and unassigned dbaeumer Oct 3, 2019
@alexr00 alexr00 modified the milestones: Backlog, October 2019 Oct 9, 2019
@alexr00
Copy link
Member

alexr00 commented Oct 28, 2019

To verify, open a folder that only has one task in it. Execute the Configure Task command. Verify that the tasks.json is opened to that one tasks and that no additional quick pick choose a task is shown.

@alexr00 alexr00 added the verification-needed Verification of issue is requested label Oct 28, 2019
@roblourens
Copy link
Member

I am seeing a quick pick blink after running that command, which is confusing, can that be avoided?

Oct-29-2019 15-14-35

@roblourens roblourens reopened this Oct 29, 2019
@roblourens roblourens added verification-found Issue verification failed and removed verification-needed Verification of issue is requested labels Oct 29, 2019
@alexr00
Copy link
Member

alexr00 commented Oct 30, 2019

It can't be avoided without changes to the quick pick (@chrmarti) since we need to pass a promise for the quick pick items into the quick pick,. If I just pass an array of quick pick items into the quick pick then in cases where there are multiple tasks there is zero UI feedback that the tasks are loading.

@alexr00 alexr00 closed this as completed Oct 30, 2019
@chrmarti
Copy link
Contributor

@alexr00 You could keep the old QuickPick around until you know how many items you have and let that indicate progress. Currently you would have to use the new interfaces to show progress in that existing QuickPick: https://github.com/Microsoft/vscode/blame/master/src/vs/platform/quickinput/common/quickInput.ts#L142

@alexr00
Copy link
Member

alexr00 commented Oct 30, 2019

Good pointer, thank you!

@alexr00
Copy link
Member

alexr00 commented Oct 30, 2019

Will make the change with #83648

@alexr00 alexr00 closed this as completed Oct 30, 2019
@alexr00 alexr00 added verification-needed Verification of issue is requested and removed verification-found Issue verification failed labels Oct 30, 2019
@eamodio
Copy link
Contributor

eamodio commented Oct 31, 2019

Looks good.

@eamodio eamodio removed the verification-needed Verification of issue is requested label Oct 31, 2019
@eamodio eamodio added the verified Verification succeeded label Oct 31, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Dec 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality tasks Task system issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

6 participants