selfstore 1.8.12 → 1.8.13

package/dist/widgets.d.ts

@@ -656,10 +656,6 @@ declare class SelfstoreDestinationElement extends FlowWidget {
     /** Stop saving to this destination. The local data stays, and so does the
      *  backup already written: this detaches, it never deletes. */
     private detach;
-    /** The connect journey, in place. Built like the gate builds it, so the
-     *  destinations, the password step and the cancel path are the library's
-     *  tested ones rather than a second implementation. */
-    private connectView;
     protected view(into: HTMLElement): void;
 }
 

package/dist/widgets.js

@@ -3717,7 +3717,7 @@ var EN8 = {
   "destination.exported": "The copy is written.",
   "destination.exportCancelled": "Nothing was written.",
   "destination.restore": "Restore a copy",
-  "destination.change": "Change destination",
+  "destination.change": "Change backup",
   "destination.detach": "Stop saving here",
   "destination.detached": "Stopped. Your data stays on this device.",
   "destination.back": "Back",
@@ -3760,7 +3760,6 @@ var SelfstoreDestinationElement = class extends FlowWidget {
   #targets = null;
   #account = null;
   #confirm = null;
-  #changing = false;
   #busy = false;
   #message = null;
   defaults() {
@@ -3850,37 +3849,9 @@ var SelfstoreDestinationElement = class extends FlowWidget {
       return this.t("destination.detached");
     });
   }
-  /** The connect journey, in place. Built like the gate builds it, so the
-   *  destinations, the password step and the cancel path are the library's
-   *  tested ones rather than a second implementation. */
-  connectView(into) {
-    const el = document.createElement(
-      siblingTag(this.localName, "destination", "connect")
-    );
-    el.store = this.store;
-    el.targets = this.#targets;
-    el.icons = this.icons;
-    el.labels = this.labels;
-    const done = () => {
-      this.#changing = false;
-      this.rerender();
-    };
-    el.addEventListener("selfstore-connected", done);
-    el.addEventListener("selfstore-cancelled", done);
-    put(
-      into,
-      h(
-        "button",
-        { part: "link destination-back", type: "button", onclick: done },
-        this.t("destination.back")
-      ),
-      el
-    );
-  }
   view(into) {
     const host = this.hostOf();
     if (!host) return;
-    if (this.#changing && this.#targets) return this.connectView(into);
     const { targetKind, label, encrypted } = host.engine.state;
     const kind = this.t(`destination.kind.${targetKind}`);
     const title = kind === `destination.kind.${targetKind}` ? label ?? targetKind : kind;
@@ -3941,10 +3912,12 @@ var SelfstoreDestinationElement = class extends FlowWidget {
           "destination-restore",
           () => this.emit("selfstore-destination-action", { action: "restore" })
         ),
-        this.#targets ? act("destination.change", "destination-change", () => {
-          this.#changing = true;
-          this.rerender();
-        }) : null,
+        // Changing backup lets the destination go and stops there: the
+        // first-run screen the app already mounts is what asks where to go
+        // next. A second connect journey inside this panel gave the same
+        // decision two mechanics depending on which control was pressed, and
+        // only one of them could be reached from anywhere in the app.
+        this.#targets ? act("destination.change", "destination-change", () => void this.detach(label)) : null,
         act("destination.detach", "destination-detach", () => void this.detach(label))
       ),
       this.#message ? h("div", { part: "sub destination-message", role: "status" }, this.#message) : null
@@ -4193,14 +4166,27 @@ var ACCOUNT_STYLES = `
 	text-overflow: ellipsis;
 	white-space: nowrap;
 }
+/* A menu item is a plain line of text, not a link: underlined blue links in a
+   menu read as four different destinations to weigh, which is exactly the
+   feeling this menu exists to remove. */
 [part~='account-item'] {
 	text-align: start;
-	padding: 0.4rem 0.6rem;
+	padding: 0.5rem 0.55rem;
+	border: none;
+	background: none;
 	border-radius: calc(var(--_radius) * 0.75);
+	font: inherit;
+	font-size: 0.8125rem;
 	text-decoration: none;
 	color: inherit;
+	cursor: pointer;
+	white-space: nowrap;
 }
 [part~='account-item']:hover { background: color-mix(in srgb, currentColor 7%, transparent); }
+/* Letting a destination go is the one gesture here that takes something away:
+   it wears the colour every other screen uses for that, and nothing else does. */
+[part~='account-change'] { color: var(--_danger); }
+[part~='account-change']:hover { background: color-mix(in srgb, var(--_danger) 10%, transparent); }
 [part~='account-sep'] { height: 1px; background: var(--_border); margin: 0.2rem 0.3rem; }
 `;
 var TICK_MS = 3e4;
@@ -4385,7 +4371,7 @@ var SelfstoreAccountElement = class extends FlowWidget {
       h(
         "button",
         {
-          part: "link account-item",
+          part: "account-item",
           type: "button",
           role: "menuitem",
           onclick: () => this.settings()
@@ -4395,7 +4381,7 @@ var SelfstoreAccountElement = class extends FlowWidget {
       h(
         "button",
         {
-          part: "link account-item account-change",
+          part: "account-item account-change",
           type: "button",
           role: "menuitem",
           onclick: () => void this.change()