Fix "fm_open_in_explorer" in sidebar causes an exception (#79)

* Fix "fm_open_in_explorer" in sidebar causes an exception

"visible_on_platforms" should be a optional arg.

Signed-off-by: Jack Cherng <jfcherng@gmail.com>

* Use "visible_on_platforms" for both sidebar and command palette

Signed-off-by: Jack Cherng <jfcherng@gmail.com>

* Make black happy

Signed-off-by: Jack Cherng <jfcherng@gmail.com>

Author: jfcherng

Side Bar.sublime-menu

@@ -79,19 +79,19 @@
     },
     {
         "caption": "Open in Explorer",
-        "platform": "!OSX",
         "command": "fm_open_in_explorer",
         "mnemonic": "O",
         "args": {
+            "visible_on_platforms": ["windows", "linux"],
             "paths": []
         }
     },
     {
         "caption": "Open in Finder",
-        "platform": "OSX",
         "command": "fm_open_in_explorer",
         "mnemonic": "O",
         "args": {
+            "visible_on_platforms": ["osx"],
             "paths": []
         }
     },

commands/open_in_explorer.py

@@ -4,7 +4,7 @@
 
 
 class FmOpenInExplorerCommand(AppCommand):
-    def run(self, visible_on_platforms, paths=None):
+    def run(self, visible_on_platforms=None, paths=None):
         # visible_on_platforms is just used by is_visible
         self.window = get_window()
         self.view = get_view()
@@ -21,5 +21,7 @@ def run(self, visible_on_platforms, paths=None):
                     {"dir": os.path.dirname(path), "file": os.path.basename(path)},
                 )
 
-    def is_visible(self, visible_on_platforms, paths=None):
-        return sublime.platform() in visible_on_platforms and super().is_visible()
+    def is_visible(self, visible_on_platforms=None, paths=None):
+        return super().is_visible() and (
+            visible_on_platforms is None or sublime.platform() in visible_on_platforms
+        )