(ctrl/exe) Clean up UX and code

code/services-core/control-service/src/main/resources/templates/control/node/node-storage-details.hdb

@@ -96,7 +96,7 @@
                     <form method="post" action="/nodes/{{node.id}}/storage/{{storage.id}}/delete" onsubmit="return confirm('Confirm deletion of {{storage.path}}')">
                         <tr>
                             <td>Delete this data</td>
-                            <td><button class="btn btn-primary" type="submit">Delete</button></td>
+                            <td><button class="btn btn-danger" type="submit">Delete</button></td>
                         </tr>
                     </form>
                 {{/if}}

code/services-core/control-service/src/main/resources/templates/control/node/node-storage-list.hdb

@@ -104,6 +104,13 @@
                     </tr>
                 {{/each}}
             {{/each}}
+            {{#unless storage}}
+                <tr>
+                    <td align="center" class="mb-3">
+                        There is nothing here!
+                    </td>
+                </tr>
+            {{/unless}}
         </table>
         <script language="javascript">
             function setActive(fileId) {
@@ -113,15 +120,15 @@
             window.location.href=window.location.href;
         }
         xhr.send();
-    }
-    function setArchived(fileId) {
-        var xhr = new XMLHttpRequest();
-        xhr.open("POST", "/nodes/{{node.id}}/storage/"+fileId+"/disable");
-        xhr.onload = () => {
-            window.location.href=window.location.href;
         }
-        xhr.send();
-    }
+        function setArchived(fileId) {
+            var xhr = new XMLHttpRequest();
+            xhr.open("POST", "/nodes/{{node.id}}/storage/"+fileId+"/disable");
+            xhr.onload = () => {
+                window.location.href=window.location.href;
+            }
+            xhr.send();
+        }
         </script>
 
         {{#if view.specs}}
@@ -132,6 +139,7 @@
                to be able to specify which domains to crawl with more control.
             </p>
             <p>The <em>ACTIVE</em> status does nothing for crawl specs.</p>
+            <p><a href="new-specs">Create New Crawl Spec</p>
             </div>
         {{/if}}
         {{#if view.crawl}}

code/services-core/executor-service/src/main/java/nu/marginalia/actor/task/CrawlActor.java

@@ -26,7 +26,6 @@ public class CrawlActor extends RecordActorPrototype {
 
     private final MqOutbox mqCrawlerOutbox;
     private final FileStorageService storageService;
-    private final Gson gson;
     private final Logger logger = LoggerFactory.getLogger(getClass());
 
     private final ActorProcessWatcher processWatcher;
@@ -84,7 +83,6 @@ public class CrawlActor extends RecordActorPrototype {
         super(gson);
         this.mqCrawlerOutbox = processOutboxes.getCrawlerOutbox();
         this.storageService = storageService;
-        this.gson = gson;
         this.processWatcher = processWatcher;
     }
 

code/services-core/executor-service/src/main/java/nu/marginalia/actor/task/RecrawlActor.java

@@ -25,7 +25,6 @@ public class RecrawlActor extends RecordActorPrototype {
     private final MqOutbox mqCrawlerOutbox;
     private final FileStorageService storageService;
     private final DomainListRefreshService refreshService;
-    private final Gson gson;
     private final ActorProcessWatcher processWatcher;
 
 
@@ -81,7 +80,6 @@ public class RecrawlActor extends RecordActorPrototype {
         this.mqCrawlerOutbox = processOutboxes.getCrawlerOutbox();
         this.storageService = storageService;
         this.refreshService = refreshService;
-        this.gson = gson;
     }
 
 }