Skip to content
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

Bug: Plugins don't activate when "Display Plugin Controls" is unchecked #458

Closed
wottpal opened this issue Mar 14, 2016 · 1 comment
Closed

Comments

@wottpal
Copy link

wottpal commented Mar 14, 2016

Hey,
I've used Atom for quite a while now and wanted to try the minimap-plugin. I've unchecked "Display Plugin Controls" because I like everything to be as minimalistic as possible. Then I've tried to install several plugins to extend the functionality of minimap:

  • minimap-find-and-replace
  • minimap-hide
  • minimap-linter

The installation of all of them finished successfully and I even restarted Atom but none of them worked. So it took some time but I figured out how to solve the problem:

  1. Enable "Display Plugin Controls" in minimaps-settings & restart Atom -> Plugin works as expected now
  2. Now you can disable "Display Plugin Controls" again & restart Atom and the plugin will continue to work as expected but without the settings-icon on hover.

This is reproducable and I have to do it for every new plugin :/ Hopefully you can figure out what is wrong with that or maybe we can figure out together if my configuration causes the problem in some way.

Dennis

@abe33
Copy link
Contributor

abe33 commented Mar 14, 2016

Hi @dkerzig, thanks for the report, I clearly overlooked that case in the tests, this will be fixed with the next version.

The main issue is that when the setting is turned on, plugins activation routine creates a setting for it with true as the default value, and the minimap plugins manager was expecting it when determining whether to activate a plugin or not. The proper behavior would be for the minimap to ignore the plugin's activation setting when Display Plugin Controls is turned off, and systematically activate the plugin.

@abe33 abe33 closed this as completed in 966eb29 Mar 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants