Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Add something like disableKeymaps: true to package.json schema. #112

Open
danielbayley opened this issue Feb 7, 2016 · 0 comments
Open

Comments

@danielbayley
Copy link

Currently, keymaps can be disabled per package by adding the package-name to packagesWithKeymapsDisabled in ATOM_HOME/config.cson (which propagates to the corresponding checkbox in settings view).

Ideally, package authors would be able to include default keybindings under keymaps/, but easily have them disabled by default with [something like] disableKeymaps: true in package.json.

Or alternatively, perhaps Atom should just disable them by default internally, without adding anything to the schema?

Existing solutions to this aren't ideal; as far as I can tell there are currently 2 options…

  1. Don't include bindings in keymaps/ and instead just suggest them in the README.
  2. Run an install:/postinstall: script from package.json to manually add package-name to packagesWithKeymapsDisabled which is a bunch of extra code for such a small thing. (This can't be done from the main package code, since it would disable the package keymaps on every load… rendering the checkbox useless.)

See also #82.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant