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 console shows even if externalConsole is true in 0.10.10 #4045

Closed
cmrigney opened this issue Mar 11, 2016 · 13 comments
Closed

Debug console shows even if externalConsole is true in 0.10.10 #4045

cmrigney opened this issue Mar 11, 2016 · 13 comments
Assignees
Labels
debug Debug viewlet, configurations, breakpoints, adapter issues feature-request Request for new features or functionality verified Verification succeeded
Milestone

Comments

@cmrigney
Copy link

In my launch.json of my node.js project, I have "externalConsole": true and so it correctly opens in a new command window. But in 0.10.10 now the embedded debug console also opens, with nothing output to it. The expected behavior is that it shouldn't be shown at all.

@weinand weinand added the debug Debug viewlet, configurations, breakpoints, adapter issues label Mar 13, 2016
@weinand weinand assigned weinand and isidorn and unassigned weinand Mar 13, 2016
@weinand
Copy link
Contributor

weinand commented Mar 14, 2016

@codyrigney92 the internal debug console is not only a console but a REPL too. This does not change if you enable the external console: the internal console is still used as the REPL. So having it open still makes sense.
But I agree, that there are situations where you do not want to see the internal debug console at all.
(BTW, this behavior hasn't changed in 0.10.10, it was always like that.)

@isidorn maybe we should consider to introduce a setting for this.

@cmrigney
Copy link
Author

@weinand I see. Makes sense. Yes I think a setting for that would be useful.

@isidorn
Copy link
Contributor

isidorn commented Mar 14, 2016

@weinand I would introduce a setting if more users ask for it

@isidorn isidorn added the feature-request Request for new features or functionality label Mar 14, 2016
@isidorn isidorn added this to the Backlog milestone Mar 14, 2016
@isidorn isidorn removed their assignment Mar 14, 2016
@Spown
Copy link

Spown commented Mar 16, 2016

how about making it consistent with task's "showOutput" setting? "always"/"never"/"silent" but "DEBUG"/"REPL"/"ALL"/"NONE" for "externalConsole"? IDK how backward compatible it would be however. Depends on weather "externalConsole" expects strictly Boolean or just !=false. But anyway I find it also annoyng that if I'm asking for external console I still get internal also.

@isidorn
Copy link
Contributor

isidorn commented Mar 17, 2016

Not sure about that, I am more leaning towards making it a user / workspace setting, and not a part of launch.json. VSCode does not interpreet the values in launch.json they are always passed to the debugger which lives outisde of vscode, and hiding the console is very vscode specific behavior.

@Spown
Copy link

Spown commented Mar 18, 2016

anyway I think I speak for everyone who sets "externalConsole" to true that unless there is something to output the internal console should remain hidden. Because (externalConsole===true) == (internalConsole===false).

I use externalConsole to maximize my editing area. It's also one of the most defining features of VSC - the interface isn't littered with all sorts of palettes, miniwindows, panels, popus, dialogs and such like in the major IDE's - only the essentials. So when an empty console view pops up when I explicitly said I want an external one - I kinda get the feeling I'm again using eclipse or MVS.

So maybe before (if at all) a new setting is added we can get a hotfix for internal console appearing only after a line was actually output (in case of externalConsole===true)?

@isidorn
Copy link
Contributor

isidorn commented Mar 18, 2016

I like the points you are making. I will address this issue for our april release, that is I will look into it next week.

@isidorn isidorn modified the milestones: April 2016, Backlog Mar 18, 2016
@isidorn
Copy link
Contributor

isidorn commented Apr 28, 2016

For april we changed a bit when we show the internal console, we only show it on the very first debug launch, not on every. So it is less spammy.
We would like to get feedback on that behavior and then in may we will introduce a setting or try another heuristic, thus moving this to may

@isidorn isidorn modified the milestones: May 2016, April 2016 Apr 28, 2016
@cmrigney
Copy link
Author

@isidorn That sounds like good behavior. I'll be happy to provide feedback.

@isidorn isidorn closed this as completed in 4726b29 May 4, 2016
@isidorn
Copy link
Contributor

isidorn commented May 4, 2016

We have now introduced an option in the launch.json to control this behavior

internalConsoleOptions which can have the following three values 'neverOpen', 'openOnSessionStart', 'openOnFirstSessionStart'

@cmrigney
Copy link
Author

cmrigney commented May 4, 2016

@isidorn Awesome thank you!

@Spown
Copy link

Spown commented May 5, 2016

thnaks!

I assume it would still open with neverOpen if I init something session unrelated like a task?

@isidorn
Copy link
Contributor

isidorn commented May 6, 2016

Yes if you start a task that has output the output panel will get opened.
But the debug console in that case would never get automatically opened.

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 verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants