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

[IMP] charts: better handling of charts with two Y axis

Open hokolomopo opened this issue 5 months ago • 1 comments

Description:

[IMP] charts: show single grid line for y-axis

When a chart had 2 vertical axis (left & right), we showed grid lines for both, which made the chart messy. Now we show only the grid lines for the left axis (or the right axis if there is no left axis).

[IMP] chart: multiple Y axis value formatting

Some chart support multiple Y axis, but the format of the values in the ticks/tooltips was the same for both charts.

With this commit, each axis can have a different format, and this format is applied to the ticks, the tooltips and the "show values" option.

Task: 4153935

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

hokolomopo avatar Sep 23 '24 07:09 hokolomopo