@fileverse-dev/dsheet 4.1.1-edit-mode-16 → 4.1.1-edit-mode-17

package/dist/editor/components/permission-chip-model.d.ts

@@ -17,7 +17,7 @@ export declare const canShowEditElevation: ({ mode, onEnterEdit, }: {
     mode: DSheetPermissionMode;
     onEnterEdit?: () => void;
 }) => boolean;
-export declare const canShowViewerModeMenu: ({ mode, onViewerModeChange, }: {
+export declare const canShowViewerModeMenu: ({ onViewerModeChange, }: {
     mode: DSheetPermissionMode;
     onViewerModeChange?: (mode: "comment" | "view") => void;
 }) => boolean;

package/dist/{index-DQvYqAJz.js → index-Ce3dAPLg.js}

@@ -50566,7 +50566,7 @@ const XT = ({
     }, Y.readAsText(b);
   });
 }, IZ = async (e, t, l, i) => {
-  const { handleExportToXLSX: n } = await import("./xlsx-export-impl-D2YNoxJm.js");
+  const { handleExportToXLSX: n } = await import("./xlsx-export-impl-CDhUELQf.js");
   return n(e, t, l, i);
 }, Hr = (e) => {
   Cy({
@@ -50658,7 +50658,7 @@ const XT = ({
   const [o, a] = Xe(!1);
   return { handleXLSXUpload: We(
     async (r, m, h) => {
-      const { runXlsxFileUpload: b } = await import("./use-xlsx-import-impl-DjiPGIoS.js");
+      const { runXlsxFileUpload: b } = await import("./use-xlsx-import-impl-mGGhmhwn.js");
       return b(
         {
           sheetEditorRef: e,
@@ -50762,14 +50762,13 @@ const XT = ({
   mode: e,
   onEnterEdit: t
 }) => e !== "edit" && !!t, yX = ({
-  mode: e,
-  onViewerModeChange: t
-}) => e !== "edit" && !!t, xT = ({
+  onViewerModeChange: e
+}) => !!e, xT = ({
   mode: e,
   onEnterEdit: t,
   onSignInToComment: l,
   onViewerModeChange: i
-}) => e === "view" && !!l && !IX({ mode: e, onEnterEdit: t }) && !yX({ mode: e, onViewerModeChange: i }), LT = ({
+}) => e === "view" && !!l && !IX({ mode: e, onEnterEdit: t }) && !yX({ onViewerModeChange: i }), LT = ({
   allowComments: e,
   isReadOnly: t,
   permissionMode: l
@@ -50779,26 +50778,25 @@ const XT = ({
   onSignInToComment: l,
   onViewerModeChange: i
 }) => {
-  const { icon: n, label: c, modifier: s } = ST(e), o = `dsheet-chip dsheet-chip--${s} inline-flex items-center py-1 gap-1 px-2 bg-yellow-100 rounded-full`, a = IX({ mode: e, onEnterEdit: t }), u = yX({
-    mode: e,
+  const { icon: n, label: c, modifier: s } = ST(e), o = `dsheet-chip dsheet-chip--${s} inline-flex items-center gap-1.5 px-2 py-1.5 rounded-md color-bg-brand-light color-text-on-brand text-xs font-medium whitespace-nowrap`, a = `${o} hover:opacity-90 transition-opacity`, u = IX({ mode: e, onEnterEdit: t }), r = yX({
     onViewerModeChange: i
-  }), r = xT({
+  }), m = xT({
     mode: e,
     onEnterEdit: t,
     onSignInToComment: l,
     onViewerModeChange: i
-  }), m = /* @__PURE__ */ w(Vt, { children: [
-    /* @__PURE__ */ d(ze, { name: n, className: "w-4 h-4 color-text-default" }),
+  }), h = /* @__PURE__ */ w(Vt, { children: [
+    /* @__PURE__ */ d(ze, { name: n, size: "sm" }),
     /* @__PURE__ */ d(
       "span",
       {
-        className: "dsheet-text dsheet-text--chip text-xs color-text-default",
+        className: "dsheet-text dsheet-text--chip",
         "data-testid": "permission-chip-label",
         children: c
       }
     )
   ] });
-  return a || u ? /* @__PURE__ */ d(
+  return u || r ? /* @__PURE__ */ d(
     ia,
     {
       align: "end",
@@ -50807,51 +50805,54 @@ const XT = ({
         "button",
         {
           type: "button",
-          className: o,
+          className: a,
           "data-testid": "permission-chip",
-          "aria-label": u ? "Switch viewer mode" : `${c}. Edit access available`,
+          "aria-label": `Switch spreadsheet mode. Current mode: ${c}`,
           children: [
-            m,
-            /* @__PURE__ */ d(
-              ze,
-              {
-                name: "ChevronDown",
-                className: "w-4 h-4 color-text-default"
-              }
-            )
+            h,
+            /* @__PURE__ */ d(ze, { name: "ChevronDown", size: "sm" })
           ]
         }
       ),
-      content: /* @__PURE__ */ w("div", { className: "flex flex-col gap-1 p-2 w-[280px] shadow-elevation-3", children: [
-        a && /* @__PURE__ */ w(
+      content: /* @__PURE__ */ w("div", { className: "flex w-[260px] flex-col gap-1 p-2 shadow-elevation-3", children: [
+        t && /* @__PURE__ */ w(
           "button",
           {
             type: "button",
-            className: "flex items-start gap-3 px-3 py-2 rounded-md text-left hover:color-bg-default-hover transition-colors",
+            "aria-pressed": e === "edit",
+            className: `flex items-center gap-2 rounded-md p-2 text-left transition-colors hover:color-bg-default-hover ${e === "edit" ? "color-bg-default-hover" : ""}`,
             "data-testid": "permission-chip-edit-option",
             onClick: t,
             children: [
               /* @__PURE__ */ d(
                 ze,
                 {
-                  name: "SquarePen",
-                  size: "md",
-                  className: "mt-0.5 shrink-0 color-text-default"
+                  name: "Pencil",
+                  size: "sm",
+                  className: "shrink-0 color-text-default"
                 }
               ),
-              /* @__PURE__ */ w("span", { className: "flex-1 min-w-0", children: [
-                /* @__PURE__ */ d("span", { className: "block text-body-sm-bold color-text-default", children: "Edit" }),
-                /* @__PURE__ */ d("span", { className: "block text-helper-text-sm color-text-secondary", children: "Make changes directly" })
-              ] })
+              /* @__PURE__ */ w("span", { className: "min-w-0 flex-1", children: [
+                /* @__PURE__ */ d("span", { className: "block text-body-sm color-text-default", children: "Edit" }),
+                /* @__PURE__ */ d("span", { className: "block text-helper-text-sm color-text-secondary", children: "Edit spreadsheet" })
+              ] }),
+              e === "edit" && /* @__PURE__ */ d(
+                ze,
+                {
+                  name: "Check",
+                  size: "sm",
+                  className: "shrink-0 color-text-default"
+                }
+              )
             ]
           }
         ),
-        u && /* @__PURE__ */ w(Vt, { children: [
+        r && /* @__PURE__ */ w(Vt, { children: [
           /* @__PURE__ */ w(
             "button",
             {
               type: "button",
-              className: "flex items-start gap-3 px-3 py-2 rounded-md text-left hover:color-bg-default-hover transition-colors",
+              className: `flex items-center gap-2 rounded-md p-2 text-left transition-colors hover:color-bg-default-hover ${e === "comment" ? "color-bg-default-hover" : ""}`,
               "data-testid": "permission-chip-comment-option",
               "aria-pressed": e === "comment",
               onClick: () => i == null ? void 0 : i("comment"),
@@ -50860,12 +50861,22 @@ const XT = ({
                   ze,
                   {
                     name: "MessageSquareText",
-                    size: "md",
-                    className: "mt-0.5 shrink-0 color-text-default"
+                    size: "sm",
+                    className: "shrink-0 color-text-default"
                   }
                 ),
-                /* @__PURE__ */ d("span", { className: "flex-1 min-w-0", children: /* @__PURE__ */ d("span", { className: "block text-body-sm-bold color-text-default", children: "View and comment" }) }),
-                e === "comment" && /* @__PURE__ */ d(ze, { name: "Check", size: "sm" })
+                /* @__PURE__ */ w("span", { className: "min-w-0 flex-1", children: [
+                  /* @__PURE__ */ d("span", { className: "block text-body-sm color-text-default", children: "View and comment" }),
+                  /* @__PURE__ */ d("span", { className: "block text-helper-text-sm color-text-secondary", children: "View and add comments" })
+                ] }),
+                e === "comment" && /* @__PURE__ */ d(
+                  ze,
+                  {
+                    name: "Check",
+                    size: "sm",
+                    className: "shrink-0 color-text-default"
+                  }
+                )
               ]
             }
           ),
@@ -50873,7 +50884,7 @@ const XT = ({
             "button",
             {
               type: "button",
-              className: "flex items-start gap-3 px-3 py-2 rounded-md text-left hover:color-bg-default-hover transition-colors",
+              className: `flex items-center gap-2 rounded-md p-2 text-left transition-colors hover:color-bg-default-hover ${e === "view" ? "color-bg-default-hover" : ""}`,
               "data-testid": "permission-chip-view-option",
               "aria-pressed": e === "view",
               onClick: () => i == null ? void 0 : i("view"),
@@ -50882,45 +50893,49 @@ const XT = ({
                   ze,
                   {
                     name: "Eye",
-                    size: "md",
-                    className: "mt-0.5 shrink-0 color-text-default"
+                    size: "sm",
+                    className: "shrink-0 color-text-default"
                   }
                 ),
-                /* @__PURE__ */ d("span", { className: "flex-1 min-w-0", children: /* @__PURE__ */ d("span", { className: "block text-body-sm-bold color-text-default", children: "View only" }) }),
-                e === "view" && /* @__PURE__ */ d(ze, { name: "Check", size: "sm" })
+                /* @__PURE__ */ w("span", { className: "min-w-0 flex-1", children: [
+                  /* @__PURE__ */ d("span", { className: "block text-body-sm color-text-default", children: "View only" }),
+                  /* @__PURE__ */ d("span", { className: "block text-helper-text-sm color-text-secondary", children: "View this spreadsheet, distraction-free" })
+                ] }),
+                e === "view" && /* @__PURE__ */ d(
+                  ze,
+                  {
+                    name: "Check",
+                    size: "sm",
+                    className: "shrink-0 color-text-default"
+                  }
+                )
               ]
             }
           )
         ] })
       ] })
     }
-  ) : r ? /* @__PURE__ */ w(
+  ) : m ? /* @__PURE__ */ w(
     "button",
     {
       type: "button",
-      className: o,
+      className: a,
       "data-testid": "permission-chip",
       "aria-label": "Sign in to comment",
       onClick: l,
       children: [
-        /* @__PURE__ */ d(
-          ze,
-          {
-            name: "MessageSquareText",
-            className: "w-4 h-4 color-text-default"
-          }
-        ),
+        /* @__PURE__ */ d(ze, { name: "MessageSquareText", size: "sm" }),
         /* @__PURE__ */ d(
           "span",
           {
-            className: "dsheet-text dsheet-text--chip text-xs color-text-default",
+            className: "dsheet-text dsheet-text--chip",
             "data-testid": "permission-chip-label",
             children: "Sign in to comment"
           }
         )
       ]
     }
-  ) : /* @__PURE__ */ d("div", { className: o, "data-testid": "permission-chip", role: "status", children: m });
+  ) : /* @__PURE__ */ d("div", { className: o, "data-testid": "permission-chip", role: "status", children: h });
 };
 function CT(e, t) {
   for (var l = -1, i = e == null ? 0 : e.length; ++l < i && t(e[l], l, e) !== !1; )
@@ -56121,7 +56136,7 @@ async function pO(e, t) {
         { suppressUiWarnings: !0 }
       );
     else {
-      const { runXlsxFileUpload: m } = await import("./use-xlsx-import-impl-DjiPGIoS.js");
+      const { runXlsxFileUpload: m } = await import("./use-xlsx-import-impl-mGGhmhwn.js");
       await m(
         {
           sheetEditorRef: s,

package/dist/index.es.js

@@ -1,4 +1,4 @@
-import { C as s, d as o, f as t, S as r, g as l, h as d, j as n, T as i, k as S, l as c, n as h, o as m, p, q as C, t as u, u as E, v as F, w as g, x, z as R, A as f, B as I, D as L, E as w, F as A, G as y, H as T, I as v, J as M, K as D, L as b, M as k, N as V } from "./index-DQvYqAJz.js";
+import { C as s, d as o, f as t, S as r, g as l, h as d, j as n, T as i, k as S, l as c, n as h, o as m, p, q as C, t as u, u as E, v as F, w as g, x, z as R, A as f, B as I, D as L, E as w, F as A, G as y, H as T, I as v, J as M, K as D, L as b, M as k, N as V } from "./index-Ce3dAPLg.js";
 import { cw as G, cx as K, cy as P, cE as U, bq as X, e as _, f as j, l as B, a as H, dg as O, dh as q, b9 as J, di as N, dj as Y, dn as Z, dp as Q, dr as W, I as $, dG as ee, ed as ae, b2 as se } from "./executeStringFunction-CxsiamWG.js";
 import { FLVURL as te } from "@fileverse-dev/formulajs";
 import { E as le, S as de } from "./constants-B3rfAvvz.js";