selfstore 1.8.12 → 1.8.14

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

@@ -2066,13 +2066,19 @@ var EN4 = {
   "destination.kind.file": "A file on this device",
   "destination.kind.webdav": "Your own server",
   "destination.kind.s3": "Your own bucket",
-  "destination.kind.device": "This device only"
+  "destination.kind.device": "This device only",
+  // A browser that cannot hold a writable file handle: the data is kept here
+  // and a backup is written only when the user asks for one. It is a mode the
+  // STORE itself falls back to, so the library owes it a name - without one, a
+  // panel showed the raw key to the user.
+  "destination.kind.file-manual": "A file you download"
 };
 var FR4 = {
   "destination.kind.file": "Un fichier sur cet appareil",
   "destination.kind.webdav": "Votre serveur",
   "destination.kind.s3": "Votre bucket",
-  "destination.kind.device": "Cet appareil seulement"
+  "destination.kind.device": "Cet appareil seulement",
+  "destination.kind.file-manual": "Un fichier, par t\xE9l\xE9chargement"
 };
 
 // src/widgets/status.ts
@@ -3717,7 +3723,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 +3766,6 @@ var SelfstoreDestinationElement = class extends FlowWidget {
   #targets = null;
   #account = null;
   #confirm = null;
-  #changing = false;
   #busy = false;
   #message = null;
   defaults() {
@@ -3850,37 +3855,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 +3918,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
@@ -4096,7 +4075,8 @@ var EN9 = {
   "account.settings": "Settings",
   "account.change": "Change backup",
   "account.saved": "Saved {when}",
-  "account.justNow": "just now"
+  "account.justNow": "just now",
+  "error.generic": "Something went wrong."
 };
 var FR9 = {
   ...FR4,
@@ -4104,7 +4084,8 @@ var FR9 = {
   "account.settings": "Param\xE8tres",
   "account.change": "Changer de sauvegarde",
   "account.saved": "Enregistr\xE9 {when}",
-  "account.justNow": "\xE0 l'instant"
+  "account.justNow": "\xE0 l'instant",
+  "error.generic": "Quelque chose n'a pas fonctionn\xE9."
 };
 var ACCOUNT_STYLES = `
 :host { position: relative; display: inline-block; }
@@ -4193,20 +4174,35 @@ 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; }
+[part~='account-error'] { margin: 0.15rem 0.55rem 0.3rem; font-size: 0.8125rem; color: var(--_danger); }
 `;
 var TICK_MS = 3e4;
 var SelfstoreAccountElement = class extends FlowWidget {
   #confirm = null;
   #open = false;
+  #error = null;
   #timer = null;
   #away = null;
   defaults() {
@@ -4327,13 +4323,20 @@ var SelfstoreAccountElement = class extends FlowWidget {
     const host = this.hostOf();
     if (!host) return;
     this.open = false;
+    this.#error = null;
     const label = host.engine.state.label;
     try {
       if (this.#confirm && !await this.#confirm({ type: "detach", label })) return;
     } catch {
       return;
     }
-    await host.engine.detachTarget();
+    try {
+      await host.engine.detachTarget();
+    } catch {
+      this.#error = this.t("error.generic");
+      this.open = true;
+      this.rerender();
+    }
   }
   card(host) {
     const { targetKind } = host.engine.state;
@@ -4385,7 +4388,7 @@ var SelfstoreAccountElement = class extends FlowWidget {
       h(
         "button",
         {
-          part: "link account-item",
+          part: "account-item",
           type: "button",
           role: "menuitem",
           onclick: () => this.settings()
@@ -4395,13 +4398,14 @@ 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()
         },
         this.t("account.change")
-      )
+      ),
+      this.#error ? h("p", { part: "account-error", role: "alert" }, this.#error) : null
     ) : null;
     put(into, trigger, menu);
     if (menu) this.anchor(menu);