@HenryGarle Bug fix for certain combinations making a mess of comma separators (#17)

* Added target-restore

target-restore reimplements the old functionality of the target list but now shows any items hidden by the default UI in the platform warning colour instead of green.

* Updated readme

* Accidental copy and paste

* Renamed file, updated main function name and updated meta data

* Bug fix for certain combinations making a mess of comma separators

* Fix a bad merge

Author: HenryGarle

README.md

@@ -173,4 +173,4 @@ This project follows the [all-contributors](https://github.com/all-contributors/
 
 > This project is not affiliated, associated, authorized, endorsed by, or in any way officially connected with Cryptohopper.com, or any of its subsidiaries or its affiliates.
 
-> The Cryptohopper name as well as related marks, emblems and images are registered trademarks of Cryptohopper.com.
+> The Cryptohopper name as well as related marks, emblems and images are registered trademarks of Cryptohopper.com.

target-restore.user.js

@@ -1,7 +1,7 @@
 // ==UserScript==
 // @name         Cryptohopper Target Restore
 // @namespace    https://github.com/markrickert/cryptohopper-dashboard-watchlist
-// @version      0.1
+// @version      0.2
 // @description  Replicates old target functionality but marks any targets that would otherwise be hidden as "inactive" and displays them in the platforms warning colour
 // @author       @henrygarle
 // @homepage     https://github.com/markrickert/cryptohopper-dashboard-watchlist
@@ -17,7 +17,6 @@
     const response = JSON.parse(xhr.responseText);
     if (response.data && response.data.ta_values) {
       const { new_target, ta_values } = response.data;
-
       // KVP of targets, true/false donates "active"
       // "Active" targets are those which the default UI would still show even after the change
       let targets = {};
@@ -39,21 +38,23 @@
 
       let targetKeys = Object.keys(targets);
       if (targetKeys.length > 0) {
-        let inactiveOutput = "";
-        let activeOutput = "";
+
+      	let inactiveTargets = [];
+      	let activeTargets = [];
+
         targetKeys.forEach((target, index) => {
           if (targets[target]) {
-            activeOutput += `${target}`;
-            if (index < targetKeys.length - 1) activeOutput += ", ";
+          	activeTargets.push(target);
           } else {
-            inactiveOutput += `${target}`;
-            if (index < targetKeys.length - 1) inactiveOutput += ", ";
+          	inactiveTargets.push(target)
           }
         });
 
-        let spinnerClass = activeOutput ? "text-success" : "text-warning";
+        let activeOutput = activeTargets.length ? activeTargets.reduce((acc, x, index) => inactiveTargets.length && index == activeTargets.length - 1 ? `${acc}, ${x}, ` : `${acc}, ${x}`) : "";
+        let inactiveOutput = inactiveTargets.length ? inactiveTargets.reduce((acc, x, index) => `${acc}, ${x}`) : "";
 
         const output = `${activeOutput}<span class="text-warning">${inactiveOutput}</span>`;
+        let spinnerClass = activeOutput ? "text-success" : "text-warning";
 
         // Set the target list
         jQuery("#current_target_coin")