feat: hide the pin/unpin option when its not applicable

Author: devvaannsh

src/command/DefaultMenus.js

@@ -32,7 +32,8 @@ define(function (require, exports, module) {
         Menus           = require("command/Menus"),
         Strings         = require("strings"),
         MainViewManager = require("view/MainViewManager"),
-        CommandManager  = require("command/CommandManager");
+        CommandManager  = require("command/CommandManager"),
+        WorkingSetView  = require("project/WorkingSetView");
 
     /**
      * Disables menu items present in items if enabled is true.
@@ -68,6 +69,14 @@ define(function (require, exports, module) {
                     Commands.NAVIGATE_OPEN_IN_DEFAULT_APP]);
             });
         }
+
+        // Pin/Unpin logic: to hide the option when its not applicable
+        const contextFile = WorkingSetView.getContext();
+        if (contextFile) {
+            const isPinned = MainViewManager.isPathPinned(MainViewManager.ACTIVE_PANE, contextFile.fullPath);
+            CommandManager.get(Commands.FILE_PIN).setEnabled(!isPinned);
+            CommandManager.get(Commands.FILE_UNPIN).setEnabled(isPinned);
+        }
     }
 
     const isBrowser = !Phoenix.isNativeApp;
@@ -320,8 +329,8 @@ define(function (require, exports, module) {
             subMenu.addMenuItem(Commands.NAVIGATE_OPEN_IN_DEFAULT_APP);
         }
         workingset_cmenu.addMenuDivider();
-        workingset_cmenu.addMenuItem(Commands.FILE_PIN);
-        workingset_cmenu.addMenuItem(Commands.FILE_UNPIN);
+        workingset_cmenu.addMenuItem(Commands.FILE_PIN, null, null, null, { hideWhenCommandDisabled: true });
+        workingset_cmenu.addMenuItem(Commands.FILE_UNPIN, null, null, null, { hideWhenCommandDisabled: true });
         workingset_cmenu.addMenuDivider();
         workingset_cmenu.addMenuItem(Commands.FILE_COPY);
         workingset_cmenu.addMenuItem(Commands.FILE_COPY_PATH);