Move visual manual tool selection to setMedia, where it belongs!

Author: techninja

resources/scripts/cncserver.client.commander.js

@@ -79,10 +79,6 @@ cncserver.cmd = {
         cncserver.api.pen.move(next[1], cncserver.cmd.cb);
         break;
       case "tool":
-        // Reflect color selection
-        $('#tools .selected').removeClass('selected');
-        $('#' + next[1]).addClass('selected');
-
         cncserver.wcb.setMedia(next[1], cncserver.cmd.cb);
         break;
       case "up":

resources/scripts/cncserver.client.wcb.js

@@ -100,6 +100,11 @@ cncserver.wcb = {
     // Save the targeted media (separate from media state)
     cncserver.state.mediaTarget = toolName;
 
+    // Visually show the selection
+    var idName = toolName.indexOf('dip') !== -1 ? toolName.slice(0, -3) : toolName;
+    $('nav#tools a.selected').removeClass('selected');
+    $('nav#tools #' + idName).addClass('selected');
+
     cncserver.wcb.status('Putting some ' + name + ' on the brush...');
     cncserver.api.tools.change(toolName, function(d){
       cncserver.wcb.status(['There is now ' + name + ' on the brush'], d);