Skip to content

fix: donut summary truncation - #967

Merged
madelineluke merged 4 commits into
mainfrom
fix/donut-summary-truncation
Oct 6, 2026
Merged

madelineluke merged 4 commits into
mainfrom
fix/donut-summary-truncation

Conversation

@madelineluke

Copy link
Copy Markdown
Collaborator

Description

If a line in the DonutSummary is going to fully truncate to ..., hide it

Related Issue

Motivation and Context

Do not show value if it fully truncates down to ...

How Has This Been Tested?

Unit tests, storybook

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@sonarqubecloud

sonarqubecloud Bot commented Oct 2, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

@madelineluke
madelineluke merged commit 1c73c49 into main Oct 6, 2026
7 checks passed
@madelineluke
madelineluke deleted the fix/donut-summary-truncation branch October 6, 2026 17:59
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