odoo/o-spreadsheet#7987

Created by fw-bot

label

odoo:master-saas-19.1-fix-side-panel-tab-save-scroll-state-rmbh-500878-fw

head

e0b20081cb3d9ba02b7b9ddabb1cca2454016f90

merged

1 week ago by Ronakkumar Mukeshbhai Bharadiya (rmbh)

	odoo/o-spreadsheet
saas-19.1	#7914
saas-19.2	#7985
master	#7987

[FIX] side_panel: scroll state issue in chart and pivot side panel

Description:

This PR includes the following fixes:
[FIX] side_panel: restore vertical scrolling for chart side panel tabs
[FIX] side_panel: restore scroll position on pivot tab change

Current behavior before PR:
- Commit d04cafc9288bd2df01f6b62abea1e267ddaf873d accidentally removed overflow-y-auto from the chart side panel content, causing the whole panel to scroll and breaking the sticky tab behavior.
- In the pivot side panel, the scrollTop value was stored when switching tabs but never reapplied to the DOM.

Desired behavior:
- Restore overflow-y-auto on the chart side panel content so the tab header stays sticky while the content scrolls.
- Apply the saved scrollTop value when switching pivot side panel tabs using a useEffect hook.

Task: 5900595

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

Forward-Port-Of: #7985
Forward-Port-Of: #7914