Skip to content

[ENHANCEMENT] PieChart: radius configuration added - #823

Open
dchromik wants to merge 2 commits into
perses:mainfrom
dchromik:feature/piechart-radius-configuration
Open

dchromik wants to merge 2 commits into
perses:mainfrom
dchromik:feature/piechart-radius-configuration

Conversation

@dchromik

Copy link
Copy Markdown

Description

Relates to perses/perses#4426

  • Changed radius to inner and outer radius, inner is optional
  • Radius was hard coded at 50 but ignored in FE where 90% was used, new default value is outer radius 90% and no inner radius
  • Moved radii and color palette to Visual group
  • Added migration from flat radius and color palette
  • Looks to me like piechart was copied from time series chart, there are some unused values, marked them as deprecated as deleting them would mean a breaking change

Screenshots

image

Checklist

  • Pull request has a descriptive title and context useful to a reviewer.
  • Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the
    following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, DOC,IGNORE.
  • All commits have DCO signoffs.

UI Changes

  • Changes that impact the UI include screenshots and/or screencasts of the relevant changes.
  • Code follows the UI guidelines.

@dchromik
dchromik requested review from a team, AntoineThebaud and Nexucis as code owners September 18, 2026 10:23
@dchromik
dchromik requested review from jgbernalp and removed request for a team September 18, 2026 10:23
Signed-off-by: Daniel Chromik <dchromik@redhat.com>
@dchromik
dchromik force-pushed the feature/piechart-radius-configuration branch from 09ce278 to ba08567 Compare September 18, 2026 13:39
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.

1 participant