update target mark logic for vehicle follow

Author: rossengeorgiev

js/app.js

@@ -385,8 +385,6 @@ $(window).ready(function() {
     $('#main').on('click', '.row .data', function() {
         var e = $(this).parent();
         followVehicle(parseInt(e.attr('class').match(/vehicle(\d+)/)[1]));
-        $('#main .row.follow').removeClass('follow');
-        e.addClass('follow');
     });
 
     // expand/collapse data when header is clicked
@@ -402,7 +400,6 @@ $(window).ready(function() {
             // disable following only we are collapsing the followed vehicle
             if(follow_vehicle == parseInt(e.attr('class').match(/vehicle(\d+)/)[1])) {
                 stopFollow();
-                e.removeClass('follow');
             }
         } else {
             // expand data for selected vehicle
@@ -419,8 +416,6 @@ $(window).ready(function() {
 
             // pan to selected vehicle
             followVehicle(parseInt(e.attr('class').match(/vehicle(\d+)/)[1]));
-            $('#main .row.follow').removeClass('follow');
-            e.addClass('follow');
         }
     });
 

js/tracker.js

@@ -369,6 +369,9 @@ function updateZoom() {
 
 function stopFollow() {
 	if(follow_vehicle != -1) {
+        // remove target mark
+        $("#main .row.follow").removeClass("follow");
+
         vehicles[follow_vehicle].follow = false;
         follow_vehicle = -1;
 
@@ -387,6 +390,11 @@ function followVehicle(index) {
     if(follow_vehicle != index) {
 		follow_vehicle = index;
 		vehicles[follow_vehicle].follow = true;
+
+        // add target mark
+        $("#main .row.follow").removeClass("follow");
+        $("#main .vehicle"+follow_vehicle).addClass("follow");
+
         updateGraph(index, true);
 	}