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