[FIX] composer: keep edition mode inactive on cursor selection change

Before this commit:
- Changing the composer cursor selection triggered a switch to edition mode,
  even when the composer was inactive.
- This caused unwanted activation of editing while simply moving the selection.

After this commit:
- Selection changes no longer modify the edition mode when the composer is
  inactive, ensuring the edition state remains stable.

closes #7523

Task: 5354541
Signed-off-by: Rémi Rahir (rar) <rar@odoo.com>

Author: rmbh-odoo

src/plugins/ui_stateful/edition.ts

@@ -144,7 +144,9 @@ export class EditionPlugin extends UIPlugin {
       case "CHANGE_COMPOSER_CURSOR_SELECTION":
         this.selectionStart = cmd.start;
         this.selectionEnd = cmd.end;
-        this.mode = "editing";
+        if (this.isSelectingForComposer()) {
+          this.mode = "editing";
+        }
         break;
       case "STOP_COMPOSER_RANGE_SELECTION":
         if (this.isSelectingForComposer()) {

tests/composer/composer_integration_component.test.ts

@@ -27,6 +27,7 @@ import {
   getElComputedStyle,
   gridMouseEvent,
   keyDown,
+  keyUp,
   rightClickCell,
   selectColumnByClicking,
   simulateClick,
@@ -339,6 +340,15 @@ describe("Composer interactions", () => {
     expect(model.getters.getEditionMode()).toBe("inactive");
   });
 
+  test("should switch topbar composer from editing to inactive when pressing Escape on cell A1 containing '=A2'", async () => {
+    setCellContent(model, "A1", "=A2");
+    await click(fixture, ".o-spreadsheet-topbar .o-composer");
+    expect(model.getters.getEditionMode()).toBe("editing");
+    keyDown({ key: "Escape" });
+    keyUp({ key: "Escape" });
+    expect(model.getters.getEditionMode()).toBe("inactive");
+  });
+
   test("ArrowKeys will move to neighbour cell, if not in contentFocus mode (left/right)", async () => {
     let composerEl: Element;
     composerEl = await startComposition("a");

tests/composer/edition_plugin.test.ts

@@ -103,6 +103,13 @@ describe("edition", () => {
     expect(model.getters.getEditionMode()).toBe("inactive");
   });
 
+  test("should keep edition mode inactive when selection changes while composer is inactive", () => {
+    const model = new Model();
+    expect(model.getters.getEditionMode()).toBe("inactive");
+    model.dispatch("CHANGE_COMPOSER_CURSOR_SELECTION", { start: 0, end: 0 });
+    expect(model.getters.getEditionMode()).toBe("inactive");
+  });
+
   test("should switch to editing mode when composer cursor selection changes", () => {
     const model = new Model();
     model.dispatch("START_EDITION", { text: "=sum(" });