Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@sandy081
Copy link
Member

@sandy081 sandy081 commented Oct 31, 2023

Motivation: My setup contains custom titlebar without command center and tabs hidden and I want to have debug tool bar on the custom title bar.

image

Floating debug tool bar is allowed to be placed anywhere in the two rows below the title bar. I enabled this to also be placed on the custom title bar. Please check the following gif.

Recording 2023-10-31 at 14 29 46

Also note that, I have shortened the tool bar when tabs and command center are hidden to match the titlebar height. In general I liked the shortened version. So, how about shortening the debug tool bar always?

CC @isidorn

@sandy081 sandy081 requested a review from roblourens October 31, 2023 13:36
@sandy081 sandy081 self-assigned this Oct 31, 2023
@vscodenpa vscodenpa added this to the November 2023 milestone Oct 31, 2023
@isidorn
Copy link
Collaborator

isidorn commented Oct 31, 2023

I like these changes overall 👏

+1 for shortening the debug toolbar all the time

fyi @connor4312

src/vs/workbench/contrib/debug/browser/debugToolBar.ts Outdated Show resolved Hide resolved
@connor4312
Copy link
Member

nice! One comment

@sandy081
Copy link
Member Author

@connor4312 Addressed the feedback

@sandy081 sandy081 enabled auto-merge October 31, 2023 15:30
@sandy081 sandy081 merged commit 9732347 into main Oct 31, 2023
@sandy081 sandy081 deleted the sandy081/floating-debug-toolbar branch October 31, 2023 15:54
@github-actions github-actions bot locked and limited conversation to collaborators Dec 15, 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.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.