[FIX] Top_bar: Opening a menu should close the composer

The composer would not close its assistant when it lost focus, which was
confusing, specifically when opening a topbar menu for instance.
A proper fix f behaviour would require some refactoring of the edition
plugin so in stable, we propose to close the composer when opening a
topbar menu.

closes #2237

Task: 3076218
X-original-commit: 07aa9f7
Signed-off-by: Rémi Rahir (rar) <rar@odoo.com>

src/components/top_bar/top_bar.ts

@@ -410,6 +410,7 @@ export class TopBar extends Component<Props, SpreadsheetChildEnv> {
     this.state.menuState.parentMenu = menu;
     this.isSelectingMenu = true;
     this.openedEl = ev.target as HTMLElement;
+    this.env.model.dispatch("STOP_EDITION");
   }
 
   closeMenus() {

tests/components/spreadsheet.test.ts

@@ -404,6 +404,15 @@ describe("Composer interactions", () => {
     fixture.remove();
     app.destroy();
   });
+
+  test("The composer helper should be closed on toggle topbar context menu", async () => {
+    await typeInComposerGrid("=sum(");
+    expect(parent.model.getters.getEditionMode()).not.toBe("inactive");
+    expect(fixture.querySelectorAll(".o-composer-assistant")).toHaveLength(1);
+    await simulateClick(".o-topbar-topleft .o-topbar-menu");
+    expect(parent.model.getters.getEditionMode()).toBe("inactive");
+    expect(fixture.querySelectorAll(".o-composer-assistant")).toHaveLength(0);
+  });
 });
 
 describe("Composer / selectionInput interactions", () => {

tests/components/top_bar.test.ts

@@ -864,3 +864,13 @@ describe("Topbar - menu item resizing with viewport", () => {
     app.destroy();
   });
 });
+
+test("The composer helper should be closed on toggle topbar context menu", async () => {
+  const { parent } = await mountSpreadsheet(fixture);
+  await typeInComposerTopBar("=sum(");
+  expect(parent.model.getters.getEditionMode()).not.toBe("inactive");
+  expect(fixture.querySelectorAll(".o-composer-assistant")).toHaveLength(1);
+  await simulateClick(".o-topbar-topleft .o-topbar-menu");
+  expect(parent.model.getters.getEditionMode()).toBe("inactive");
+  expect(fixture.querySelectorAll(".o-composer-assistant")).toHaveLength(0);
+});