Skip to content

Conversation

hokolomopo
Copy link
Contributor

Description

This commit slightly improves the looks of bar charts by reducing the spacing between bars and rounding the bar corners.

Task: 5165821

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 Oct 14, 2025

Pull request status dashboard

@hokolomopo hokolomopo force-pushed the master-sexier-bar-chart-adrm branch from 0b99e4f to 6d77a2f Compare October 15, 2025 10:47
@hokolomopo hokolomopo force-pushed the master-sexier-bar-chart-adrm branch 2 times, most recently from bd33bef to d8e1879 Compare October 23, 2025 07:51
This commit slightly improves the looks of bar charts by reducing
the spacing between bars and rounding the bar corners.

Task: 5165821
@hokolomopo hokolomopo force-pushed the master-sexier-bar-chart-adrm branch from d8e1879 to d3dbf36 Compare October 23, 2025 07:54
@rrahir
Copy link
Collaborator

rrahir commented Oct 23, 2025

robodoo r+

robodoo pushed a commit that referenced this pull request Oct 23, 2025
This commit slightly improves the looks of bar charts by reducing
the spacing between bars and rounding the bar corners.

closes #7325

Task: 5165821
Signed-off-by: Rémi Rahir (rar) <rar@odoo.com>
@robodoo robodoo added the 19.1 label Oct 23, 2025
@robodoo robodoo closed this Oct 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants