fix(window): Pin/unpin icon should show the respective action not window's state telerik/kendo#7059

Author: jivanova

src/kendo.window.js

@@ -373,7 +373,7 @@
                 var windowSpecificCommands = [ "maximize", "minimize" ];
 
                 actions = $.map(actions, function(action) {
-                    action = !pinned && action.toLowerCase() === "pin" ? "unpin" : action;
+                    action = pinned && action.toLowerCase() === "pin" ? "unpin" : action;
                     return { name: (windowSpecificCommands.indexOf(action.toLowerCase()) > - 1) ? "window-" + action : action };
                 });
 
@@ -547,8 +547,8 @@
                     "k-i-window-minimize": "minimize",
                     "k-i-window-restore": "restore",
                     "k-i-refresh": "refresh",
-                    "k-i-pin": "unpin",
-                    "k-i-unpin": "pin"
+                    "k-i-pin": "pin",
+                    "k-i-unpin": "unpin"
                 }[iconClass];
             },
 
@@ -1070,7 +1070,7 @@
 
                 if (!that.options.isMaximized) {
                     wrapper.css({position: "fixed", top: top - win.scrollTop(), left: left - win.scrollLeft()});
-                    wrapper.children(KWINDOWTITLEBAR).find(KUNPIN).addClass("k-i-pin").removeClass("k-i-unpin");
+                    wrapper.children(KWINDOWTITLEBAR).find(KPIN).addClass("k-i-unpin").removeClass("k-i-pin");
 
                     that.options.pinned = true;
                     that.options.draggable = false;
@@ -1086,7 +1086,7 @@
 
                 if (!that.options.isMaximized) {
                     wrapper.css({position: "", top: top + win.scrollTop(), left: left + win.scrollLeft()});
-                    wrapper.children(KWINDOWTITLEBAR).find(KPIN).addClass("k-i-unpin").removeClass("k-i-pin");
+                    wrapper.children(KWINDOWTITLEBAR).find(KUNPIN).addClass("k-i-pin").removeClass("k-i-unpin");
 
                     that.options.pinned = false;
                     that.options.draggable = true;

tests/window/api.js

@@ -546,13 +546,13 @@
             actions: ["Pin"]
         });
 
-        equal(dialog.wrapper.find(".k-i-unpin").length, 1);
-        equal(dialog.wrapper.find(".k-i-pin").length, 0);
+        equal(dialog.wrapper.find(".k-i-pin").length, 1);
+        equal(dialog.wrapper.find(".k-i-unpin").length, 0);
 
         dialog.pin();
 
-        equal(dialog.wrapper.find(".k-i-pin").length, 1);
-        equal(dialog.wrapper.find(".k-i-unpin").length, 0);
+        equal(dialog.wrapper.find(".k-i-unpin").length, 1);
+        equal(dialog.wrapper.find(".k-i-pin").length, 0);
     });
 
     test("pin() substracts the browser scroll position from the wrapper's top and left styles", function() {
@@ -604,13 +604,13 @@
             pinned: true
         });
 
-        equal(dialog.wrapper.find(".k-i-pin").length, 1);
-        equal(dialog.wrapper.find(".k-i-unpin").length, 0);
+        equal(dialog.wrapper.find(".k-i-unpin").length, 1);
+        equal(dialog.wrapper.find(".k-i-pin").length, 0);
 
         dialog.unpin();
 
-        equal(dialog.wrapper.find(".k-i-unpin").length, 1);
-        equal(dialog.wrapper.find(".k-i-pin").length, 0);
+        equal(dialog.wrapper.find(".k-i-pin").length, 1);
+        equal(dialog.wrapper.find(".k-i-unpin").length, 0);
     });
 
     test("unpin() adds the browser scroll position to the wrapper's top and left styles", function() {
@@ -777,7 +777,7 @@
         dialog.maximize();
         dialog.restore();
 
-        equal(dialog.wrapper.find(".k-i-unpin:visible").length, 1);
+        equal(dialog.wrapper.find(".k-i-pin:visible").length, 1);
     });
 
     test("restore() removes the k-window-maximized class", function() {

tests/window/initialization.js

@@ -420,7 +420,7 @@
 
     test("creating pinned window with a Pin command adds an Unpin button", function() {
         var dialog = createWindow({ pinned: true, actions: ["Pin"] });
-        equal(dialog.wrapper.find(".k-i-pin").length, 1);
+        equal(dialog.wrapper.find(".k-i-unpin").length, 1);
     });
 
     test("creating window with an int zero top position applies a top style", function() {