[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 #2250

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

src/components/top_bar/top_bar.ts

@@ -362,6 +362,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

@@ -205,6 +205,15 @@ describe("Simple Spreadsheet Component", () => {
     const sheets = document.querySelectorAll(".o-all-sheets .o-sheet");
     expect(sheets).toHaveLength(model.getters.getSheetIds().length - 1);
   });
+
+  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);
+  });
 });
 
 test("Can instantiate a spreadsheet with a given client id-name", async () => {

tests/components/top_bar.test.ts

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