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

Make revealing debug console on debug start a configuration #5889

Closed
v-pavanp opened this issue Apr 27, 2016 · 13 comments
Closed

Make revealing debug console on debug start a configuration #5889

v-pavanp opened this issue Apr 27, 2016 · 13 comments
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s) feature-request Request for new features or functionality
Milestone

Comments

@v-pavanp
Copy link

  • VSCode Version: 1.1.0-alpha
  • OS Version: WInodws10

Steps to Reproduce:

  1. Launch app.
  2. Set configuration for debugging : task.json, launch.json and tsconfig.json.
  3. Set "Showoutput" property 'always' to get message in the output window.
  4. Start debugging and verify debug window.
  5. Now perform build operation with CTRL+Shift+B and verify output window.
  6. Start debugging again and verify debug console window.

Actual: Debug console window is not showing up when user start debugging after build.
Expected: Debug console window should be on top when user start debugging.

outoutwindowissue

@bpasero
Copy link
Member

bpasero commented Apr 27, 2016

I think we got smarter here about when to show the debug console. @isidorn please chime in

@isidorn
Copy link
Contributor

isidorn commented Apr 27, 2016

We only reveal the debug console on the first debug session start, not on consecutive ones.
We are no longer forcing the debug console on every debug start because that was too obtrusive for some debug users.
Closing as designed as we would like to get feedback from users for this change. If users prefere that we always show the debug console, then we can easily go back to our old behavior, but for now I would really like to get more feeback.

@isidorn isidorn closed this as completed Apr 27, 2016
@v-pavanp
Copy link
Author

Can we make it configurable?

@isidorn
Copy link
Contributor

isidorn commented Apr 28, 2016

Yes, that's a good idea. Let me reopen this and rename the issue.

@isidorn isidorn reopened this Apr 28, 2016
@isidorn isidorn added the feature-request Request for new features or functionality label Apr 28, 2016
@isidorn isidorn changed the title Output and debug console window should swap according to user's action. Make revealing debug console on debug start a configuration Apr 28, 2016
@isidorn isidorn added this to the May 2016 milestone Apr 28, 2016
@isidorn
Copy link
Contributor

isidorn commented May 4, 2016

@bpasero how does debug.openConsoleOnStart sound to you? Default value would be false

@bpasero
Copy link
Member

bpasero commented May 4, 2016

@isidorn maybe this should be a setting within a launch config instead of global?

@isidorn
Copy link
Contributor

isidorn commented May 4, 2016

@bpasero also fine by me. I think that actually makes more sense since you might want different behavior per configuration. Like in your case, launching unit tests vs launching app. If we put it inside launch.json I would just call it openConsoleOnStart

@weinand opinions?

@weinand
Copy link
Contributor

weinand commented May 4, 2016

@isidorn some comments:

  • since this will only apply to the internal console, we probably should use 'internal' in the attribute name.
  • instead of making the attribute boolean, I suggest to make it more open ended like Dirk's showOutput. It is very likely that there will be more settings in the future. I suggest to call the attribute internalConsoleOptions and use a value of openOnSessionStart or revealOnSessionStart for the case considered here.

@bpasero
Copy link
Member

bpasero commented May 4, 2016

Oh yeah, I was actually thinking about how it is possible for tasks to control this and was expecting something similar.

@isidorn isidorn closed this as completed in 4726b29 May 4, 2016
@isidorn isidorn added the *duplicate Issue identified as a duplicate of another issue(s) label May 4, 2016
@isidorn
Copy link
Contributor

isidorn commented May 4, 2016

I applied the changes you proposed and explained them in #4045

@v-pavanp
Copy link
Author

v-pavanp commented May 9, 2016

Thanks for the updates, I verified and getting debug console window all the time with my setting. But I observed that application behavior got changed for task window. I used to get task window all the time whenever I performed CTRL+SHIFT+B (build command) but now task window is not displaying all the time. With the latest alpha build I observed that user needs to explicitly open task window to see the build output. Did we also make the task window configurable? If so what are the settings I can use to get it all the time for build action.
Basically my intention here is that console windows (task and debug) should be interchangeable with user's action and of course according to local setting.

@isidorn
Copy link
Contributor

isidorn commented May 9, 2016

@dbaeumer can comment on the task window. As far as I know that is also dependent on the setting in your tasks.json

@v-pavanp
Copy link
Author

v-pavanp commented May 9, 2016

OOPS! I forgot setting in tasks.json "showOutput": "always". Now It is working flawlessly. Thanks again.

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s) feature-request Request for new features or functionality
Projects
None yet
Development

No branches or pull requests

4 participants