@3sln/nook 0.1.0 → 0.2.0

package/src/providers/onboarding.js

@@ -29,6 +29,7 @@ export function onboardingResource({ storage, owner }) {
     storage,
     hasSeen: () => resource.hasSeen(),
   });
+  resource.acknowledgeAttention = () => page.attention?.acknowledge();
   resource.subscribeAttention = (listener) =>
     page.attention.subscribe(listener);
   return resource;

package/src/ui/bounds.js

@@ -1,4 +1,5 @@
 const definitions = new Set([
+  "nook-item",
   "nook-action",
   "nook-setting",
   "nook-menu",

package/src/ui/feedback.js

@@ -12,7 +12,8 @@ export default function feedback({ view, make, i18n, dock, getHooks, signal }) {
   kind.type = "hidden";
   kind.name = "type";
   kind.value = "bug";
-  const label = make("label", "feedback-label", "Tell us more");
+  const label = make("label", "feedback-label");
+  label.append(make("span", "feedback-message-label", "Tell us more"));
   label.className = "feedback-label";
   const text = make("textarea", "feedback-message");
   text.name = "message";
@@ -104,7 +105,16 @@ export default function feedback({ view, make, i18n, dock, getHooks, signal }) {
   previewEmpty.hidden = true;
   const thumbnails = make("div", "capture-thumbnails");
   thumbnails.className = "capture-thumbnails";
-  gallery.append(largePreview, previewEmpty, thumbnails);
+  const hidePreviews = make("button", "capture-hide-previews", "Hide previews");
+  hidePreviews.type = "button";
+  hidePreviews.className = "quiet";
+  const collapsePreviews = () => {
+    gallery.hidden = true;
+    review.setAttribute("aria-expanded", "false");
+    review.focus();
+  };
+  hidePreviews.onclick = collapsePreviews;
+  gallery.append(hidePreviews, largePreview, previewEmpty, thumbnails);
   controls.append(shutter, review, captureDone);
   sessionUI.append(controls, captureStatus, gallery);
   (dock.overlayAnchor || dock.anchor).append(sessionUI);
@@ -273,7 +283,8 @@ export default function feedback({ view, make, i18n, dock, getHooks, signal }) {
     if (event.key === "Escape") {
       event.preventDefault();
       event.stopPropagation();
-      finishCapture();
+      if (!gallery.hidden) collapsePreviews();
+      else finishCapture();
     }
   });
   signal.addEventListener("abort", () => finishCapture(false), { once: true });

package/src/ui/keyboardLabels.js

@@ -0,0 +1,31 @@
+// Platform affects wording only. It never establishes a keyboard layout.
+export function keyboardLabels(navigator) {
+  const platform =
+    navigator.userAgentData?.platform || navigator.platform || "";
+  const mac = /mac/i.test(platform);
+  return {
+    mac,
+    alt: mac ? "Option" : "Alt",
+    back: mac ? "Delete" : "Backspace",
+    format(binding) {
+      return binding
+        .split("+")
+        .map((key) => {
+          const value = key.trim();
+          return (
+            {
+              alt: mac ? "Option" : "Alt",
+              opt: mac ? "Option" : "Alt",
+              option: mac ? "Option" : "Alt",
+              ctrl: "Ctrl",
+              control: "Ctrl",
+              meta: mac ? "Command" : "Meta",
+              cmd: mac ? "Command" : "Meta",
+              command: mac ? "Command" : "Meta",
+            }[value.toLowerCase()] || value
+          );
+        })
+        .join(" + ");
+    },
+  };
+}