Skip to content

[FIX] charts: bubble legend hover colors#8874

Open
dhrp-odoo wants to merge 1 commit into
masterfrom
master-fix-bubble-chart-legend-hover-dhrp
Open

[FIX] charts: bubble legend hover colors#8874
dhrp-odoo wants to merge 1 commit into
masterfrom
master-fix-bubble-chart-legend-hover-dhrp

Conversation

@dhrp-odoo

Copy link
Copy Markdown
Contributor

Description:

Bubble charts with multiple colors store per-bubble colors as arrays. The legend hover callbacks reused line chart highlighting, which expects single color values and crashes when resetting highlights.

Use bubble-specific highlighting based on the original hover colors so legend hover works with per-bubble colors and preserves transparency.

Task: 6269018

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

robodoo commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

Pull request status dashboard

Bubble charts with multiple colors store per-bubble colors as arrays.
The legend hover callbacks reused line chart highlighting, which expects
single color values and crashes when resetting highlights.

Use bubble-specific highlighting based on the original hover colors so
legend hover works with per-bubble colors and preserves transparency.

Task: 6269018
@dhrp-odoo dhrp-odoo force-pushed the master-fix-bubble-chart-legend-hover-dhrp branch from 62fec14 to 074bd7a Compare June 8, 2026 04:52
@dhrp-odoo dhrp-odoo marked this pull request as ready for review June 8, 2026 04:52
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.

2 participants