-
Notifications
You must be signed in to change notification settings - Fork 29.4k
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
OS Specific Keybindings #8962
Comments
I would personally not complicate the structure for the majority of users that don't need to jump across Operating Systems, I would go for something like @bpasero @aeschli @jrieken - You are also heavy cross OS users and share the ownership of the config service. Thoughts? |
I'm hitting the same issue where I would like to sync my settings cross machines, including Windows and OSX machines. The only thing that currently is not working well for me is keybindings. Instead of scoping all of the keybindings, I agree that it makes more sense to allow creation of user-specific keybindings per platform in different files (so besides normal Just as a thought, another option is introducing a |
I've been syncing my settings between Linux and OS X and am trying to resolve an issue where I'm loading custom CSS but the path picked up in the settings file by |
We've run into a similar issue with Python .venv support, where the pythonPath differs by OS but is otherwise seems sensible to store in the repo with other things that should be shared. {
// win
"python.pythonPath": "${workspaceRoot}\\.venv\\Scripts\\python.exe",
"python.formatting.provider": "yapf"
}
{
// mac
"python.pythonPath": "${workspaceRoot}/.venv/bin/python",
"python.formatting.provider": "yapf"
} Thanks for considering! |
Here's my keybindings.json file, my dotfiles script symlinks that to the correct directory on each platform https://github.com/Tyriar/dotfiles/blob/master/modules/vscode/config/keybindings.json Some of the keybindings don't make sense on mac in particular, for example this one which makes ctrl+k a chord on macOS: { "key": "ctrl+k ctrl+s", "command": "workbench.action.openGlobalKeybindingsFile" }, Something like this would be sufficient for me without complicating the structure: { "key": "ctrl+k ctrl+s", "command": "workbench.action.openGlobalKeybindingsFile",
"when": "!isMac" }, |
Yeah isMac or an operatingSystem variable would be perfect
…On Mon, Apr 16, 2018, 06:13 Daniel Imms ***@***.***> wrote:
Here's my keybindings.json file, my dotfiles script symlinks that to the
correct directory
https://github.com/Tyriar/dotfiles/blob/master/modules/vscode/config/keybindings.json
Some of the keybindings don't make sense on mac in particular, for example
this one which makes ctrl+k a chord on macOS:
{ "key": "ctrl+k ctrl+s", "command": "workbench.action.openGlobalKeybindingsFile" },
Something like this would be sufficient for me without complicating the
structure:
{ "key": "ctrl+k ctrl+s", "command": "workbench.action.openGlobalKeybindingsFile",
"when": "!isMac" },
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#8962 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAKyr17Zy2K_NCCWgi3h8XPUf1xFZtZkks5tpJkHgaJpZM4JILlq>
.
|
I'd like for this feature to exist as well, mostly to integrate better with windows. I use vscode across Mac, Linux and Windows and my issue is with the |
Leaving settings.json as platform agnostic and making os-specific files that can be shared on all OSs for convenience, but only applied on their target platform seems like a good idea. |
Good implementation. Thanks! |
verified via code review |
I have a company issued OSX machine, linux boxes and windows machines all of which I do dev work on, moving back and forth, I try to keep my settings in sync, it would be very nice to have an operating specific syntax for the keybindings file, much how tasks.json does.
Or better yet in line with #871 ...
For that matter it would be wonderful to have this ability for all of the user configurable settings
The text was updated successfully, but these errors were encountered: