-
Notifications
You must be signed in to change notification settings - Fork 31.9k
Allow hiding of launcher on Run and Debug title bar when debugging #193812
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
Conversation
| @@ -452,7 +452,7 @@ configurationRegistry.registerConfiguration({ | |||
| }, | |||
| 'debug.toolBarLocation': { | |||
| enum: ['floating', 'docked', 'commandCenter', 'hidden'], | |||
| markdownDescription: nls.localize({ comment: ['This is the description for a setting'], key: 'toolBarLocation' }, "Controls the location of the debug toolbar. Either `floating` in all views, `docked` in the debug view, `commandCenter` (requires `{0}`), or `hidden`.", '#window.commandCenter#', '#window.titleBarStyle#'), | |||
| markdownDescription: nls.localize({ comment: ['This is the description for a setting'], key: 'toolBarLocation' }, "Controls the location of the debug toolbar. Either `floating` in all views, `docked` in the debug view, `commandCenter` (requires `{0}`), or `hidden`.", '#window.commandCenter#'), | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is cleanup of a redundant argument I spotted when copying the line for reuse in the new setting.
| @@ -588,8 +588,13 @@ configurationRegistry.registerConfiguration({ | |||
| }, | |||
| 'debug.enableStatusBarColor': { | |||
| type: 'boolean', | |||
| description: nls.localize('debug.enableStatusBarColor', "Color status bar when debugger is active"), | |||
| description: nls.localize('debug.enableStatusBarColor', "Color status bar when debugger is active."), | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixing a nit.
|
Please consider adding this to the October 2023 milestone. |
|
Sorry for the slow response @gjsjohnmurray, I'm not up to date on notifications right now. Are you saying that the problem for you is going for that button when you meant to click the 'continue' button? |
|
@roblourens yes, exactly that. |
|
Now that the 1.83.1 recovery has shipped please consider taking this |
|
@roblourens any hope of this making October before endgame starts next week? |
|
We can take this, but can you leave the default behavior the same? |
|
I have changed the setting to an opt-in and renamed it |
|
I mean, I'm fine with settings that are false by default too 😁 but we have a mix of "enable" and "disable" settings in vscode so this is fine |
Allow hiding of launcher on Run and Debug title bar when debugging
I was motivated to create this PR by the new option to put the debug toolbar in the Command Center. Even before this I often found myself clicking the green 'play' button in the title bar of the Run and Debug view when I intended to click the same button on the floating debug toolbar. With the floating toolbar I solved the problem by positioning it over that title bar, which I obviously cannot do after moving the toolbar to Command Center.
The PR adds a setting
debug.hideLauncherWhileDebuggingto hide the view title bar control when debugging, moving its functionality to the '...' menu just as already happens if the debug toolbar location is set todocked.(recording shows original setting name before it was inverted)
Pinging @roblourens for debugging and @jrieken for the new Command Center location for the debug toolbar.