@yuneta/gobj-ui 7.23.56 → 7.23.57

package/dist/gobj-ui.cjs.js

@@ -5900,18 +5900,18 @@ function refresh_clear($input) {
 *          Copyright (c) 2026, ArtGins.
 *          All Rights Reserved.
 ***********************************************************************/
-var LEFT_BUTTONS = [
-	"save",
+var LEFT_BUTTONS = ["copy", "paste"];
+var RIGHT_BUTTONS = [
 	"undo",
-	"clear"
+	"clear",
+	"save"
 ];
-var RIGHT_BUTTONS = ["copy", "paste"];
 var DEFAULT_TOOLBAR = [
-	"save",
+	"copy",
+	"paste",
 	"undo",
 	"clear",
-	"copy",
-	"paste"
+	"save"
 ];
 /***************************************************************
 *  plan_toolbar(wanted) -> {left, right, unknown}

package/dist/gobj-ui.es.js

@@ -5895,18 +5895,18 @@ function refresh_clear($input) {
 *          Copyright (c) 2026, ArtGins.
 *          All Rights Reserved.
 ***********************************************************************/
-var LEFT_BUTTONS = [
-	"save",
+var LEFT_BUTTONS = ["copy", "paste"];
+var RIGHT_BUTTONS = [
 	"undo",
-	"clear"
+	"clear",
+	"save"
 ];
-var RIGHT_BUTTONS = ["copy", "paste"];
 var DEFAULT_TOOLBAR = [
-	"save",
+	"copy",
+	"paste",
 	"undo",
 	"clear",
-	"copy",
-	"paste"
+	"save"
 ];
 /***************************************************************
 *  plan_toolbar(wanted) -> {left, right, unknown}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yuneta/gobj-ui",
-  "version": "7.23.56",
+  "version": "7.23.57",
   "type": "module",
   "main": "dist/gobj-ui.cjs.js",
   "module": "dist/gobj-ui.es.js",

package/src/form_toolbar_plan.js

@@ -10,13 +10,25 @@
  *          All Rights Reserved.
  ***********************************************************************/
 
-/*  Save/undo/clear sit on the left of the bar, copy/paste on the right.
- *  That split is what the layout has always drawn, and keeping it means
- *  a caller who drops one group does not leave a hole in the middle. */
-const LEFT_BUTTONS  = ["save", "undo", "clear"];
-const RIGHT_BUTTONS = ["copy", "paste"];
+/*  Two groups, and the split is a real one: `undo`, `clear` and `save`
+ *  act on the RECORD being edited, `copy` and `paste` move it in and out
+ *  of the clipboard. Keeping them apart means a caller who drops one
+ *  group does not leave a hole in the middle.
+ *
+ *  WHICH SIDE EACH ONE TAKES CHANGED (2026-09-04), and the reason is that
+ *  the form changed shape: it was an embedded panel and it opens as a
+ *  MODAL DIALOG now. In a dialog the primary action goes bottom-RIGHT --
+ *  every platform puts it there, and it is where the eye ends after
+ *  reading the fields. `save` sat hard against the left edge, which is
+ *  where a person looks last.
+ *
+ *  So: clipboard on the left, record actions on the right, and `save`
+ *  LAST of all. The order inside a group is the order asked for, which
+ *  is why DEFAULT_TOOLBAR is written in the order it is drawn.  */
+const LEFT_BUTTONS  = ["copy", "paste"];
+const RIGHT_BUTTONS = ["undo", "clear", "save"];
 
-const DEFAULT_TOOLBAR = ["save", "undo", "clear", "copy", "paste"];
+const DEFAULT_TOOLBAR = ["copy", "paste", "undo", "clear", "save"];
 
 
 /***************************************************************

package/src/form_toolbar_plan.test.js

@@ -5,17 +5,26 @@ describe("plan_toolbar", () => {
 
     it("says nothing -> the five buttons this gclass always had", () => {
         const p = plan_toolbar(undefined);
-        expect(p.left).toEqual(["save", "undo", "clear"]);
-        expect(p.right).toEqual(["copy", "paste"]);
+        expect(p.left).toEqual(["copy", "paste"]);
+        expect(p.right).toEqual(["undo", "clear", "save"]);
         expect(p.unknown).toEqual([]);
         expect(DEFAULT_TOOLBAR.length).toBe(5);
     });
 
+    it("save is the LAST button of the bar", () => {
+        /*  The form opens as a modal dialog and a dialog's primary action
+         *  goes bottom-right. Asserted on the plan and not on the CSS,
+         *  because the plan is what decides the two groups.  */
+        const p = plan_toolbar(undefined);
+        expect(p.right[p.right.length - 1]).toBe("save");
+        expect(p.left).not.toContain("save");
+    });
+
     it("a non-array is not an empty toolbar: it is the default", () => {
         /*  Reading a JSON attr that was never set must not silently
          *  leave a form with no save button. */
         for(const v of [null, "save", 0, {}]) {
-            expect(plan_toolbar(v).left).toContain("save");
+            expect(plan_toolbar(v).right).toContain("save");
         }
     });
 
@@ -28,21 +37,21 @@ describe("plan_toolbar", () => {
 
     it("one action: a dialog asks for save only", () => {
         const p = plan_toolbar(["save"]);
-        expect(p.left).toEqual(["save"]);
-        expect(p.right).toEqual([]);
+        expect(p.right).toEqual(["save"]);
+        expect(p.left).toEqual([]);
     });
 
     it("keeps the order asked for inside each side", () => {
         const p = plan_toolbar(["clear", "save", "paste", "copy"]);
-        expect(p.left).toEqual(["clear", "save"]);
-        expect(p.right).toEqual(["paste", "copy"]);
+        expect(p.right).toEqual(["clear", "save"]);
+        expect(p.left).toEqual(["paste", "copy"]);
     });
 
     it("an unknown name is REPORTED, never silently dropped", () => {
         /*  A typo here would otherwise remove a button and leave no
          *  trace of why it is missing. */
         const p = plan_toolbar(["save", "sav", "undo"]);
-        expect(p.left).toEqual(["save", "undo"]);
+        expect(p.right).toEqual(["save", "undo"]);
         expect(p.unknown).toEqual(["sav"]);
     });
 
@@ -59,7 +68,7 @@ describe("plan_toolbar", () => {
 
     it("drops one whole side without disturbing the other", () => {
         const p = plan_toolbar(["save", "undo", "clear"]);
-        expect(p.left).toEqual(["save", "undo", "clear"]);
-        expect(p.right).toEqual([]);
+        expect(p.right).toEqual(["save", "undo", "clear"]);
+        expect(p.left).toEqual([]);
     });
 });