odoo/o-spreadsheet#7415

Created by BI, Rémi Rahir (rar)
Merged at e053137c8a865b52d267850759a9536313271aa7

Statuses:

label
odoo:saas-18.4-fix-menubar-visibility-mobile-rar
head
a553d4464fa960556540ebd6f2963d04b2ecc3fd
merged
5 days ago by BI, Lucas Lefèvre (lul)
odoo/o-spreadsheet
saas-18.4 #7415
19.0 #7446
master #7447

[FIX] TopbarMenu: specify `isReadonlyAllowed`

The logic that displays the top-level items of the topbarmenu is isolated from the Menu component logic. More specifically, it never checks if the item can be used in readonly and as such, we never took the time to define the key isReadonlyAllowed on those menu items.

However RibbonMenu relies on the logic of Menu so in order to display the topbar menu items properly in mobile mode, we need the key isReadonlyAllowed to be defined.

Task: 5245432

review checklist

  • [ ] feature is organized in plugin, or UI components
  • [ ] support of duplicate sheet (deep copy)
  • [ ] in model/core: ranges are Range object, and can be adapted (adaptRanges)
  • [ ] in model/UI: ranges are strings (to show the user)
  • [ ] undo-able commands (uses this.history.update)
  • [ ] multiuser-able commands (has inverse commands and transformations where needed)
  • [ ] new/updated/removed commands are documented
  • [ ] exportable in excel
  • [ ] translations (_t("qmsdf %s", abc))
  • [ ] unit tested
  • [ ] clean commented code
  • [ ] track breaking changes
  • [ ] doc is rebuild (npm run doc)
  • [ ] status is correct in Odoo