search

atom / snippets

Atom snippets package
MIT License
205 stars 100 forks source link

Add an option to disable all included snippets. #216

Closed CentricStorm closed 4 years ago

CentricStorm commented 7 years ago

Add an option to disable all package snippets in one go. This is designed for users that want to start with a clean slate without having to override all of the snippets that they don't want.

50Wliu commented 7 years ago

This setting at the very least needs to be per-package, not an "all-or-nothing" toggle. You may want to see how the keymap toggle works.

CentricStorm commented 7 years ago

This setting at the very least needs to be per-package, not an "all-or-nothing" toggle.

A separate per-package setting can be added alongside this one.

For some users a per-package setting is suboptimal because:

  1. Every time a new package is added, snippets have to be disabled.
  2. Every time snippets are added to an existing package, snippets have to be disabled.

For users that want to completely disable all included snippets, both problems simply waste time and increase editor maintenance.

50Wliu commented 7 years ago

Your second point is invalid as if the per-package setting is properly implemented, new snippets provided by the package should be automatically disabled as well.

CentricStorm commented 7 years ago

Your second point is invalid as if the per-package setting is properly implemented, new snippets provided by the package should be automatically disabled as well.

If a package starts without any snippets, but then some snippets are added in an update, those snippets will have to be disabled (unless packages are disabled by default, but that doesn't seem likely).