o-spreadsheet icon indicating copy to clipboard operation
o-spreadsheet copied to clipboard

[FIX] dashboard: hide overflow area after snappy resize

Open rmbh-odoo opened this issue 1 month ago • 1 comments

Description:

Current behavior before PR:

  • The canvas was sized from the global grid instead of the overlay, causing visible white margins after the snappy resize (commit 3c8df4b9).

Desired behavior after PR is merged:

  • The dashboard applies overflow-hidden to clip the extra painted area, keeping the snappy resizing behavior.

Task: 5223156

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

rmbh-odoo avatar Nov 05 '25 06:11 rmbh-odoo

Pull request status dashboard

robodoo avatar Nov 05 '25 06:11 robodoo

if I understand the issue correctly, we miss the actual computation of the spreadsheet width when in dashboard mode. have you looked into a way to correct that as well? because right now we have a discrepency between the canvas size and the actual part displayed to the client and I'm not sure that just hiding it is the best solution (maybe it's too complicated to fix and it's the right call, but i'm curious to know if looked into it)

rrahir avatar Nov 10 '25 06:11 rrahir