@scalemule/news-publication 0.0.31 → 0.0.33

package/dist/story-review.cjs

@@ -452,7 +452,7 @@ function StoryReview({
     }
     setPanel("share");
   }
-  async function send(action, success) {
+  async function send(action, success, consumeDraft = true) {
     if (!review || busy) return;
     if (action.action === "APPROVE" && file) {
       setNotice("Your photo is not uploaded yet. Upload it or remove the selection before approving.");
@@ -464,7 +464,7 @@ function StoryReview({
     setNotice("");
     try {
       const actionPanel = { ANSWER_QUESTIONS: "questions", REQUEST_CHANGES: "changes", COMMENT: "comment", PROPOSE_EDITS: "edits", EDIT_MEDIA_METADATA: "metadata", APPROVE: "approve", DECLINE: "decline" }[String(action.action)];
-      const receipt = actionPanel ? await drafts.receipt(actionPanel) : void 0;
+      const receipt = consumeDraft && actionPanel ? await drafts.receipt(actionPanel) : void 0;
       const sent = await request("", { revision_id: review.revision.id, ...action, ...receipt ? { draft: receipt } : {} });
       if (sent.cleared_draft) drafts.consumed(sent.cleared_draft);
       try {
@@ -578,6 +578,7 @@ function StoryReview({
   const selected = revisions.find((r) => r.id === compare), selectedAfter = revisions.find(
     (r) => r.id === (compareTo || review?.revision.id)
   );
+  const errorMessage = error ? /* @__PURE__ */ jsxRuntime.jsx("p", { ref: errorRef, tabIndex: -1, role: "alert", className: "sm-review-alert", children: error }) : null;
   return /* @__PURE__ */ jsxRuntime.jsxs("div", { className: `sm-review-frame${publicationHeader ? " sm-review-publication-frame" : ""}`, children: [
     /* @__PURE__ */ jsxRuntime.jsx("div", { className: "sm-review-banner-wrap", children: /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "sm-review-private", children: [
       /* @__PURE__ */ jsxRuntime.jsx("strong", { children: review?.published ? "Private review" : "Private preview \xB7 Not published" }),
@@ -588,7 +589,7 @@ function StoryReview({
       /* @__PURE__ */ jsxRuntime.jsx("span", { children: review?.publication.name || publicationName })
     ] }) }),
     /* @__PURE__ */ jsxRuntime.jsxs("main", { className: `sm-story-review${celebration ? " sm-review-celebration" : ""}`, children: [
-      error && /* @__PURE__ */ jsxRuntime.jsx("p", { ref: errorRef, tabIndex: -1, role: "alert", className: "sm-review-alert", children: error }),
+      (!review || panel === "article" || verification) && errorMessage,
       notice && !(approved && finalEditor && notice === approvalThanks) && /* @__PURE__ */ jsxRuntime.jsx("p", { ref: noticeRef, tabIndex: -1, role: "status", className: "sm-review-notice", children: notice }),
       verification && /* @__PURE__ */ jsxRuntime.jsxs("section", { className: "sm-review-form", children: [
         /* @__PURE__ */ jsxRuntime.jsx("h2", { children: "Verify your email" }),
@@ -799,7 +800,7 @@ function StoryReview({
             className: `sm-review-actions${celebration ? " sm-review-concierge-actions" : ""}${panel !== "article" ? " sm-review-actions-editing" : ""}`,
             "aria-label": "Review actions",
             children: [
-              !can("APPROVE") && can("COMMENT") && (review.identity_required || review.participant) && /* @__PURE__ */ jsxRuntime.jsx("button", { disabled: busy, onClick: () => setPanel("feedback"), children: "Looks good to me" }),
+              !can("APPROVE") && can("COMMENT") && (review.identity_required || review.participant) && /* @__PURE__ */ jsxRuntime.jsx("button", { className: "sm-review-primary", disabled: busy, onClick: () => setPanel("feedback"), children: "Looks Good \u2014 Send Feedback" }),
               can("APPROVE") && !review.published && /* @__PURE__ */ jsxRuntime.jsx(
                 "button",
                 {
@@ -835,6 +836,7 @@ function StoryReview({
         ] }),
         panel === "share" && /* @__PURE__ */ jsxRuntime.jsxs("section", { ref: responseRef, tabIndex: -1, className: "sm-review-form", "aria-label": "Share private preview", children: [
           /* @__PURE__ */ jsxRuntime.jsx("h2", { children: "Share with your family" }),
+          errorMessage,
           /* @__PURE__ */ jsxRuntime.jsx("p", { children: "Anyone with this link can read the private preview. To send feedback or photos, they will enter their own name and verify their email. No account is needed." }),
           /* @__PURE__ */ jsxRuntime.jsx("label", { htmlFor: "review-share-url", children: "Private preview link" }),
           /* @__PURE__ */ jsxRuntime.jsx("input", { id: "review-share-url", readOnly: true, value: shareUrl, onFocus: (e) => e.currentTarget.select() }),
@@ -850,6 +852,7 @@ function StoryReview({
         ] }),
         panel === "identity" && /* @__PURE__ */ jsxRuntime.jsxs("section", { ref: responseRef, tabIndex: -1, className: "sm-review-form", "aria-label": "Introduce yourself", children: [
           /* @__PURE__ */ jsxRuntime.jsx("h2", { children: identityStep === "code" ? "Check your email" : "Who is helping with the story?" }),
+          errorMessage,
           /* @__PURE__ */ jsxRuntime.jsx("p", { children: "Your name will accompany your feedback for the newsroom. Your email stays private. No password or account is needed." }),
           identityStep === "name" ? /* @__PURE__ */ jsxRuntime.jsxs("form", { onSubmit: async (e) => {
             e.preventDefault();
@@ -912,12 +915,13 @@ function StoryReview({
         ] }),
         panel === "feedback" && /* @__PURE__ */ jsxRuntime.jsxs("section", { ref: responseRef, tabIndex: -1, className: "sm-review-form", "aria-label": "Family feedback", children: [
           /* @__PURE__ */ jsxRuntime.jsx("h2", { children: "Looks good to you?" }),
+          errorMessage,
           /* @__PURE__ */ jsxRuntime.jsxs("p", { children: [
             "We'll send your feedback as ",
             review.reviewer,
             ". The newsroom will request approval from the designated family reviewer separately. This will not publish the story."
           ] }),
-          /* @__PURE__ */ jsxRuntime.jsx("button", { disabled: busy, onClick: () => void send({ action: "COMMENT", message: "This draft looks good to me.", target: { kind: "STORY" } }, "Thank you. Your feedback has been sent to the newsroom. The story is still private."), children: "Send my feedback" }),
+          /* @__PURE__ */ jsxRuntime.jsx("button", { className: "sm-review-primary", disabled: busy, onClick: () => void send({ action: "COMMENT", message: "This draft looks good to me.", target: { kind: "STORY" } }, "Thank you. Your feedback has been sent to the newsroom. The story is still private.", false), children: "Send my feedback" }),
           /* @__PURE__ */ jsxRuntime.jsx("button", { disabled: busy, onClick: () => setPanel("article"), children: "Back to the story" })
         ] }),
         !["article", "identity", "share", "feedback"].includes(panel) && /* @__PURE__ */ jsxRuntime.jsxs(
@@ -932,6 +936,7 @@ function StoryReview({
                 /* @__PURE__ */ jsxRuntime.jsx("h2", { children: panel === "approve" ? "Approve this exact draft" : panel === "edits" ? "Propose changes" : panel === "media" ? photoToReplace ? "Change photo" : "Contribute a photograph" : panel === "questions" ? "Tell us a little more" : panel === "history" ? "Selected revision history" : panel === "decline" ? "Prefer not to publish" : panel === "changes" ? "Suggest a change" : "Message the newsroom" }),
                 /* @__PURE__ */ jsxRuntime.jsx("button", { disabled: busy, onClick: () => setPanel("article"), children: "Close" })
               ] }),
+              errorMessage,
               panel !== "history" && /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "sm-review-save-state", role: "status", "aria-live": "polite", children: [
                 currentDraft?.status === "conflict" ? /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
                   /* @__PURE__ */ jsxRuntime.jsx("p", { children: "A response was saved in another tab or device. Your writing here is still available. Choose which version to keep." }),

package/dist/story-review.js

@@ -450,7 +450,7 @@ function StoryReview({
     }
     setPanel("share");
   }
-  async function send(action, success) {
+  async function send(action, success, consumeDraft = true) {
     if (!review || busy) return;
     if (action.action === "APPROVE" && file) {
       setNotice("Your photo is not uploaded yet. Upload it or remove the selection before approving.");
@@ -462,7 +462,7 @@ function StoryReview({
     setNotice("");
     try {
       const actionPanel = { ANSWER_QUESTIONS: "questions", REQUEST_CHANGES: "changes", COMMENT: "comment", PROPOSE_EDITS: "edits", EDIT_MEDIA_METADATA: "metadata", APPROVE: "approve", DECLINE: "decline" }[String(action.action)];
-      const receipt = actionPanel ? await drafts.receipt(actionPanel) : void 0;
+      const receipt = consumeDraft && actionPanel ? await drafts.receipt(actionPanel) : void 0;
       const sent = await request("", { revision_id: review.revision.id, ...action, ...receipt ? { draft: receipt } : {} });
       if (sent.cleared_draft) drafts.consumed(sent.cleared_draft);
       try {
@@ -576,6 +576,7 @@ function StoryReview({
   const selected = revisions.find((r) => r.id === compare), selectedAfter = revisions.find(
     (r) => r.id === (compareTo || review?.revision.id)
   );
+  const errorMessage = error ? /* @__PURE__ */ jsx("p", { ref: errorRef, tabIndex: -1, role: "alert", className: "sm-review-alert", children: error }) : null;
   return /* @__PURE__ */ jsxs("div", { className: `sm-review-frame${publicationHeader ? " sm-review-publication-frame" : ""}`, children: [
     /* @__PURE__ */ jsx("div", { className: "sm-review-banner-wrap", children: /* @__PURE__ */ jsxs("div", { className: "sm-review-private", children: [
       /* @__PURE__ */ jsx("strong", { children: review?.published ? "Private review" : "Private preview \xB7 Not published" }),
@@ -586,7 +587,7 @@ function StoryReview({
       /* @__PURE__ */ jsx("span", { children: review?.publication.name || publicationName })
     ] }) }),
     /* @__PURE__ */ jsxs("main", { className: `sm-story-review${celebration ? " sm-review-celebration" : ""}`, children: [
-      error && /* @__PURE__ */ jsx("p", { ref: errorRef, tabIndex: -1, role: "alert", className: "sm-review-alert", children: error }),
+      (!review || panel === "article" || verification) && errorMessage,
       notice && !(approved && finalEditor && notice === approvalThanks) && /* @__PURE__ */ jsx("p", { ref: noticeRef, tabIndex: -1, role: "status", className: "sm-review-notice", children: notice }),
       verification && /* @__PURE__ */ jsxs("section", { className: "sm-review-form", children: [
         /* @__PURE__ */ jsx("h2", { children: "Verify your email" }),
@@ -797,7 +798,7 @@ function StoryReview({
             className: `sm-review-actions${celebration ? " sm-review-concierge-actions" : ""}${panel !== "article" ? " sm-review-actions-editing" : ""}`,
             "aria-label": "Review actions",
             children: [
-              !can("APPROVE") && can("COMMENT") && (review.identity_required || review.participant) && /* @__PURE__ */ jsx("button", { disabled: busy, onClick: () => setPanel("feedback"), children: "Looks good to me" }),
+              !can("APPROVE") && can("COMMENT") && (review.identity_required || review.participant) && /* @__PURE__ */ jsx("button", { className: "sm-review-primary", disabled: busy, onClick: () => setPanel("feedback"), children: "Looks Good \u2014 Send Feedback" }),
               can("APPROVE") && !review.published && /* @__PURE__ */ jsx(
                 "button",
                 {
@@ -833,6 +834,7 @@ function StoryReview({
         ] }),
         panel === "share" && /* @__PURE__ */ jsxs("section", { ref: responseRef, tabIndex: -1, className: "sm-review-form", "aria-label": "Share private preview", children: [
           /* @__PURE__ */ jsx("h2", { children: "Share with your family" }),
+          errorMessage,
           /* @__PURE__ */ jsx("p", { children: "Anyone with this link can read the private preview. To send feedback or photos, they will enter their own name and verify their email. No account is needed." }),
           /* @__PURE__ */ jsx("label", { htmlFor: "review-share-url", children: "Private preview link" }),
           /* @__PURE__ */ jsx("input", { id: "review-share-url", readOnly: true, value: shareUrl, onFocus: (e) => e.currentTarget.select() }),
@@ -848,6 +850,7 @@ function StoryReview({
         ] }),
         panel === "identity" && /* @__PURE__ */ jsxs("section", { ref: responseRef, tabIndex: -1, className: "sm-review-form", "aria-label": "Introduce yourself", children: [
           /* @__PURE__ */ jsx("h2", { children: identityStep === "code" ? "Check your email" : "Who is helping with the story?" }),
+          errorMessage,
           /* @__PURE__ */ jsx("p", { children: "Your name will accompany your feedback for the newsroom. Your email stays private. No password or account is needed." }),
           identityStep === "name" ? /* @__PURE__ */ jsxs("form", { onSubmit: async (e) => {
             e.preventDefault();
@@ -910,12 +913,13 @@ function StoryReview({
         ] }),
         panel === "feedback" && /* @__PURE__ */ jsxs("section", { ref: responseRef, tabIndex: -1, className: "sm-review-form", "aria-label": "Family feedback", children: [
           /* @__PURE__ */ jsx("h2", { children: "Looks good to you?" }),
+          errorMessage,
           /* @__PURE__ */ jsxs("p", { children: [
             "We'll send your feedback as ",
             review.reviewer,
             ". The newsroom will request approval from the designated family reviewer separately. This will not publish the story."
           ] }),
-          /* @__PURE__ */ jsx("button", { disabled: busy, onClick: () => void send({ action: "COMMENT", message: "This draft looks good to me.", target: { kind: "STORY" } }, "Thank you. Your feedback has been sent to the newsroom. The story is still private."), children: "Send my feedback" }),
+          /* @__PURE__ */ jsx("button", { className: "sm-review-primary", disabled: busy, onClick: () => void send({ action: "COMMENT", message: "This draft looks good to me.", target: { kind: "STORY" } }, "Thank you. Your feedback has been sent to the newsroom. The story is still private.", false), children: "Send my feedback" }),
           /* @__PURE__ */ jsx("button", { disabled: busy, onClick: () => setPanel("article"), children: "Back to the story" })
         ] }),
         !["article", "identity", "share", "feedback"].includes(panel) && /* @__PURE__ */ jsxs(
@@ -930,6 +934,7 @@ function StoryReview({
                 /* @__PURE__ */ jsx("h2", { children: panel === "approve" ? "Approve this exact draft" : panel === "edits" ? "Propose changes" : panel === "media" ? photoToReplace ? "Change photo" : "Contribute a photograph" : panel === "questions" ? "Tell us a little more" : panel === "history" ? "Selected revision history" : panel === "decline" ? "Prefer not to publish" : panel === "changes" ? "Suggest a change" : "Message the newsroom" }),
                 /* @__PURE__ */ jsx("button", { disabled: busy, onClick: () => setPanel("article"), children: "Close" })
               ] }),
+              errorMessage,
               panel !== "history" && /* @__PURE__ */ jsxs("div", { className: "sm-review-save-state", role: "status", "aria-live": "polite", children: [
                 currentDraft?.status === "conflict" ? /* @__PURE__ */ jsxs(Fragment, { children: [
                   /* @__PURE__ */ jsx("p", { children: "A response was saved in another tab or device. Your writing here is still available. Choose which version to keep." }),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@scalemule/news-publication",
-  "version": "0.0.31",
+  "version": "0.0.33",
   "description": "Shared server framework for ScaleMule local news publications. Walnut Creek Times is the broadsheet default; each paper supplies its own identity and feature flags.",
   "type": "module",
   "main": "./dist/index.cjs",

package/story-review.css

@@ -15,10 +15,15 @@
   margin: auto;
   padding-inline: 32px;
 }
-.sm-review-publication-frame .sm-review-banner-wrap {
-  width: min(var(--pub-width, 1240px), calc(100% - 2 * var(--pub-gutter, 32px)));
+.sm-review-frame .sm-review-banner-wrap {
+  width: 100%;
   max-width: none;
-  padding-inline: 0;
+  padding-inline: max(16px, calc((100% - 976px) / 2));
+  background: #fff3c4;
+  border-bottom: 2px solid #d5ac30;
+}
+.sm-review-publication-frame .sm-review-banner-wrap {
+  padding-inline: max(var(--pub-gutter, 16px), calc((100% - var(--pub-width, 1240px)) / 2));
 }
 .sm-review-publication-frame {
   padding-bottom: 100px;
@@ -31,11 +36,10 @@
   max-width: none;
 }
 .sm-review-publication-frame .sm-review-private {
-  color: var(--pub-muted, #545c55);
-  border-color: var(--pub-rule, #d7dcd7);
+  color: #5b4816;
 }
 .sm-review-publication-frame .sm-review-private strong {
-  color: var(--pub-ink, #171917);
+  color: #382700;
 }
 .sm-story-review {
   color-scheme: light;
@@ -51,8 +55,7 @@
 }
 .sm-review-private {
   padding: 14px 0;
-  border-bottom: 1px solid #d7dcd7;
-  color: var(--pub-muted, #545c55);
+  color: #5b4816;
   font:
     13px/1.5 var(--font-inter),
     Arial,
@@ -63,9 +66,9 @@
   flex-wrap: wrap;
 }
 .sm-review-private strong {
-  color: var(--pub-ink, #171917);
+  color: #382700;
   text-transform: uppercase;
-  font-size: 11px;
+  font-size: 13px;
   letter-spacing: 0.08em;
 }
 .sm-review-masthead {