Do all edits in place. Row copy buttons work but copy all does not.

main.js

@@ -1,12 +1,96 @@
 "use strict";
-document.body.style.border = "5px solid #ff0080";
-//alert("The test extension is up and running");
-console.log('main.js says hello');
 
 
-//browser.runtime.onMessage.addListener(request => {
-	//document.body.style.border = "15px solid #ff0080";
-	//console.log("Message from the background script:");
-	//console.log(request.greeting);
-	//return Promise.resolve({response: "Hi from content script"});
-//});
+function makeCopyButtons() {
+    // shopping basket items are in li tags with class "basket-item"
+    console.log('makeCopyButtons says hello');
+
+    var num_created = 0;
+
+    var copy_all_button_class = 'copy-all-button';
+    var copy_button_class = 'copy-button';
+    var basket_top = document.querySelector('.basket');
+    if (basket_top == null) {
+        // likely the page hasn't loaded yet, don't do anything this time
+        return;
+    }
+    var existing_all_butt = basket_top.parentElement.querySelectorAll('.' +  copy_all_button_class);
+    if (existing_all_butt.length == 0) {
+        num_created++;
+        var all_butt = document.createElement('BUTTON');
+        all_butt.className = copy_all_button_class;
+        all_butt.onclick = () => {
+
+            let copyStr = '| wine | url | stock | qty_val | qty_unit |'
+                        + '| --- | --- | --- | --- | --- |';
+            navigator.clipboard.writeText(copyStr);
+        }
+        all_butt.innerHTML = 'Copy all items';
+        basket_top.prepend(all_butt);
+    }
+
+    var basket_items = document.querySelectorAll('li.basket-item');
+    console.log('basket_items: ')
+    console.log(basket_items)
+    for (let bi of basket_items) {
+        var existing_butts = bi.querySelectorAll('.' +  copy_button_class);
+        if (existing_butts.length > 0) {
+            console.log('button already exists' + bi);
+            continue;
+        }
+        num_created++;
+        console.log('hello bi ' + bi);
+
+        var link_in_title = bi.querySelector('.basket-item__title a');
+        var abs_url = link_in_title.href;
+        var wine_desc = link_in_title.text;
+        var stock_status_span = bi.querySelector('.badge-label');
+        try {
+            var quantity_value_sel = bi.querySelector('.basket-item__quantity-select select');
+            var quantity_value = quantity_value_sel.value;
+            var quantity_unit_sel = bi.querySelector('.basket-item__type select');
+            var quantity_unit = quantity_unit_sel.value;
+        } catch (err) {
+            var quantity_value = '?';
+            var quantity_unit = '?';
+        }
+        var item = {
+                wine: wine_desc,
+                url: abs_url,
+                stock: stock_status_span.text,
+                qty_val: quantity_value,
+                qty_unit: quantity_unit
+        };
+        var butt = document.createElement('BUTTON');
+        butt.className = copy_button_class;
+        // I'm not sure why this mess is needed, but I was previously getting
+        // every button copying the *last* value copyString had in the loop.
+        function onclickgen(copyStr) {
+            function onclickinner() {
+                navigator.clipboard.writeText(copyStr);
+            }
+            return onclickinner;
+        }
+        let copyString = `| ${item.wine} | ${item.url} | ${item.stock} | ${item.qty_val} | ${item.qty_unit} |`;
+        butt.onclick = onclickgen(copyString);
+        butt.innerHTML = 'Copy';
+        bi.prepend(butt);
+    }
+    console.log('Created ' + num_created + ' new elements');
+
+    // cancel ourself if repeated calls have had no further effect
+    if (num_created == 0) {
+        fruitless_run_count++;
+    } else {
+        fruitless_run_count = 0;
+    }
+    if (fruitless_run_count > 10) {
+        clearInterval(intervalID);
+    }
+
+}
+
+
+var fruitless_run_count = 0;
+var intervalID = setInterval(makeCopyButtons, 1000);
+

manifest.json

@@ -16,13 +16,7 @@
 			"matches":["https://*.thewinesociety.com/*"],
 			"js":["main.js"]
 		}
-	],
-	"browser_action": {
-		"default_icon": "icons/wine32.png",
-		"default_title": "Copy wine links",
-		"default_popup": "popup/copy_wine.html"
-	}
-
+	]
 }