Skip to content

feat(workbench/window): add setting to switch TouchBar on macOS#40400

Closed
viztor wants to merge 1 commit intomicrosoft:masterfrom
viztor:feature/touchbar-setting
Closed

feat(workbench/window): add setting to switch TouchBar on macOS#40400
viztor wants to merge 1 commit intomicrosoft:masterfrom
viztor:feature/touchbar-setting

Conversation

@viztor
Copy link

@viztor viztor commented Dec 18, 2017

No description provided.

@msftclas
Copy link

msftclas commented Dec 18, 2017

CLA assistant check
All CLA requirements met.

'default': true
}
},
'workbench.touchBar.enabled': {
Copy link
Member

Choose a reason for hiding this comment

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

@viztor the setting should only be enabled on macOS, not all platforms. I also suggest to move it into the existing keyboard category:

image

E.g. call it keyboard.touchBar.enabled

import { IIntegrityService } from 'vs/platform/integrity/common/integrity';
import { AccessibilitySupport, isRootUser, isWindows, isMacintosh } from 'vs/base/common/platform';
import product from 'vs/platform/node/product';
import { ConfigurationService } from 'vs/platform/configuration/node/configurationService';
Copy link
Member

Choose a reason for hiding this comment

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

@viztor unused import

@bpasero bpasero added this to the On Deck milestone Dec 18, 2017
@bpasero bpasero modified the milestones: On Deck, December 2017/January 2018 Dec 22, 2017
@bpasero
Copy link
Member

bpasero commented Dec 22, 2017

Marking as duplicate of #40692

@bpasero bpasero closed this Dec 22, 2017
@bpasero bpasero added the *duplicate Issue identified as a duplicate of another issue(s) label Dec 22, 2017
@viztor viztor deleted the feature/touchbar-setting branch December 27, 2017 22:30
@github-actions github-actions bot locked and limited conversation to collaborators Mar 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

*duplicate Issue identified as a duplicate of another issue(s)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants