-
Notifications
You must be signed in to change notification settings - Fork 30.3k
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
Add setting to make touchbar controls optional #70174
Add setting to make touchbar controls optional #70174
Conversation
src/vs/code/electron-main/window.ts
Outdated
@@ -979,6 +979,8 @@ export class CodeWindow extends Disposable implements ICodeWindow { | |||
// of the group directly prevents ugly flickering from happening | |||
this.touchBarGroups.forEach((touchBarGroup, index) => { | |||
const commands = groups[index]; | |||
// Clear the segments first to prevent icon or label remaining | |||
touchBarGroup.segments = []; |
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.
There seems to be a bug in electron where it doesn't correctly clear icons or labels when segments are replaced (See: electron/electron#17322). This gets around it by clearing out the segments then replacing them. No noticeable flicker when tested on my machine.
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 issue has been fixed in Electron (electron/electron#17336) but has yet to be released.
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 has now been released in https://github.com/electron/electron/releases/tag/v3.1.7
What is the process for updating dependencies of VSCode? Are there any guidelines that need to be followed?
Thanks, merged with the settings name |
Related issues: #63865, #38333, on2-dev/nasc-vscode-mac-touchbar#9
Adds a new setting that allows the user to selectively have controls not appear in the touchbar without having to completely disable it.
Example settings to hide all debug controls but keep navigation:
This also allows people to use plugins that make use of the touchbar without having to also have the nav and debug controls (that take up much of the space).
Screenshots
Default:
Debug controls disabled:
Debug controls disabled with nasc-vscode-mac-touchbar extension enabled: