Skip to content

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

Merged
merged 6 commits into from
Oct 20, 2023

Conversation

gjsjohnmurray
Copy link
Contributor

@gjsjohnmurray gjsjohnmurray commented Sep 22, 2023

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.hideLauncherWhileDebugging to 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 to docked.

junk
(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.

@@ -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#'),
Copy link
Contributor Author

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."),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixing a nit.

@gjsjohnmurray
Copy link
Contributor Author

Please consider adding this to the October 2023 milestone.

@roblourens
Copy link
Member

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?

@gjsjohnmurray
Copy link
Contributor Author

@roblourens yes, exactly that.

@gjsjohnmurray
Copy link
Contributor Author

Now that the 1.83.1 recovery has shipped please consider taking this

@gjsjohnmurray
Copy link
Contributor Author

@roblourens any hope of this making October before endgame starts next week?

@roblourens roblourens added this to the October 2023 milestone Oct 20, 2023
@roblourens
Copy link
Member

We can take this, but can you leave the default behavior the same?

@gjsjohnmurray gjsjohnmurray changed the title Hide launcher on Run and Debug title bar when debugging Allow hiding of launcher on Run and Debug title bar when debugging Oct 20, 2023
@gjsjohnmurray
Copy link
Contributor Author

I have changed the setting to an opt-in and renamed it debug.hideLauncherWhileDebugging

@roblourens
Copy link
Member

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

@roblourens roblourens enabled auto-merge October 20, 2023 22:32
@roblourens roblourens merged commit b0022c8 into microsoft:main Oct 20, 2023
@gjsjohnmurray gjsjohnmurray deleted the pink-bedbug branch October 21, 2023 09:57
Alex0007 pushed a commit to Alex0007/vscode that referenced this pull request Oct 26, 2023
Allow hiding of launcher on Run and Debug title bar when debugging
@github-actions github-actions bot locked and limited conversation to collaborators Dec 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants