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

Debug Auto Attach: introduce an option to disable without hiding status bar item #111021

Closed
weinand opened this issue Nov 20, 2020 · 4 comments
Closed
Assignees
Labels
debug Debug viewlet, configurations, breakpoints, adapter issues feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@weinand
Copy link
Contributor

weinand commented Nov 20, 2020

Currently we have these Auto Attach settings:

2020-11-20_12-06-45

What I think is missing is a setting that disables Auto Attach but continues to show it in the status bar.
Sometimes I just want to disable Auto Attach temporarily but since the status bar item is gone, I forget to enable it again...

I think we have two options to achieve this:

  • we just add a new value "Off" (or "Never" if we want to match "Always") for the Auto Attach setting and leave the "Disabled" untouched, or
  • we change the semantics of the existing "Disabled" setting to not hide the status bar item. In this case we'd rely on the item's context menu to show/hide status bar items.

/cc @egamma

@weinand weinand added the debug Debug viewlet, configurations, breakpoints, adapter issues label Nov 20, 2020
@connor4312 connor4312 added this to the November 2020 milestone Nov 20, 2020
@connor4312
Copy link
Member

This has been on my mine for a little while, thus done. There's a new option that disables auto attach for the "current session", which doesn't require a reload (and will only be in effect until the window is reloaded).

recording(9)

@connor4312 connor4312 added feature-request Request for new features or functionality verification-needed Verification of issue is requested labels Nov 21, 2020
chenjigeng pushed a commit to chenjigeng/vscode that referenced this issue Nov 22, 2020
@weinand weinand added the verified Verification succeeded label Dec 3, 2020
@weinand
Copy link
Contributor Author

weinand commented Dec 3, 2020

There is a new "Temporarily Disable" option in the QuickPick.

The fact that "Temporarily Disable" is only visible if auto attach is already shown in the status bar surprised me a bit. But I think the logic behind that is ok.

More problematic is that running the previous action again, does not stop the temporary disablement.
So I had "Always" enabled, and then did a "Temporarily Disable". Later I run "Always" again, but did not notice that this is ignored. All other action would actually stop the temporary disablement.
IMO this is a bug that we should fix before shipping the feature.

@weinand weinand added verification-found Issue verification failed and removed verified Verification succeeded labels Dec 3, 2020
@weinand weinand reopened this Dec 3, 2020
@connor4312 connor4312 removed the verification-found Issue verification failed label Dec 4, 2020
@weinand weinand added the verified Verification succeeded label Dec 4, 2020
@weinand
Copy link
Contributor Author

weinand commented Dec 4, 2020

This works fine now.

But I've filed #111890

@connor4312
Copy link
Member

Agree, I've had to point that out to users a few times.

Thanks for verifying!

@github-actions github-actions bot locked and limited conversation to collaborators Jan 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
debug Debug viewlet, configurations, breakpoints, adapter issues feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants
@weinand @connor4312 and others