Skip to content

Conversation

hokolomopo
Copy link
Contributor

[IMP] menu: hide parent menu if empty

A menu item would still be displayed if all its children were hidden,
opening an empty submenu. We had to manually add a isVisible logic
to all parent menu items to avoid this.

This commit make it so that a parent menu item is automatically
hidden if all its children are hidden.

[IMP] pivot: add menu items to collapse/expand all groups

This commit adds menu item int he right click context menu to:

  • collapse/expand a pivot group
  • collapse/expand all pivot groups

Task: 4942260

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

@robodoo
Copy link
Collaborator

robodoo commented Sep 23, 2025

Pull request status dashboard

A menu item would still be displayed if all its children were hidden,
opening an empty submenu. We had to manually add a `isVisible` logic
to all parent menu items to avoid this.

This commit make it so that a parent menu item is automatically
hidden if all its children are hidden.

Task: 4942260
@hokolomopo hokolomopo force-pushed the master-pivot-collapse-menu-items-adrm branch from 85599c8 to 6ccda87 Compare September 24, 2025 13:57
@hokolomopo hokolomopo force-pushed the master-pivot-collapse-menu-items-adrm branch from 6ccda87 to 16e47fd Compare September 30, 2025 06:07
This commit adds menu item int he right click context menu to:
- collapse/expand a pivot group
- collapse/expand all pivot groups

Task: 4942260
@hokolomopo hokolomopo force-pushed the master-pivot-collapse-menu-items-adrm branch from 16e47fd to 3754f42 Compare October 2, 2025 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants