[FIX] clipboard: don't open composer on firefox

Steps to reproduce:
- open a spreadsheet with firefox
- copy a cell
- paste it somewhere else
=> the composer is open with the pasted content.

Additional small fix in the content editable helper, the selection `start`
was not clipped if it was greater than the content length.

Note: it doesn't happen on Chrome because `ev.data` is null when
pasting content. It doesn't correctly implements the spec
https://w3c.github.io/input-events/#dfn-data

opw-3236303

closes #2257

X-original-commit: c7f1486
Signed-off-by: Rémi Rahir (rar) <[email protected]>
Signed-off-by: Lucas Lefèvre (lul) <[email protected]>

src/components/composer/content_editable_helper.ts

@@ -32,6 +32,7 @@ export class ContentEditableHelper {
         );
         if (start < 0) start = 0;
         if (end > textLength) end = textLength;
+        if (start > textLength) start = textLength;
       }
       let startNode = this.findChildAtCharacterIndex(start);
       let endNode = this.findChildAtCharacterIndex(end);

src/components/grid/grid.ts

@@ -466,6 +466,10 @@ export class Grid extends Component<Props, SpreadsheetChildEnv> {
   }
 
   onInput(ev: InputEvent) {
+    // the user meant to paste in the sheet, not open the composer with the pasted content
+    if (!ev.isComposing && ev.inputType === "insertFromPaste") {
+      return;
+    }
     if (ev.data) {
       // if the user types a character on the grid, it means he wants to start composing the selected cell with that
       // character

tests/components/grid.test.ts

@@ -759,6 +759,20 @@ describe("Grid component", () => {
       await nextTick();
       expect(fixture.querySelector(".o-menu")).toBeTruthy();
     });
+
+    test("input event triggered from a paste should not open composer", async () => {
+      const input = fixture.querySelector(".o-grid>input");
+      input?.dispatchEvent(
+        new InputEvent("input", {
+          data: "d",
+          bubbles: true,
+          isComposing: false,
+          inputType: "insertFromPaste",
+        })
+      );
+      await nextTick();
+      expect(model.getters.getEditionMode()).toBe("inactive");
+    });
   });
 });