Skip to content

Conversation

Mash707
Copy link
Contributor

@Mash707 Mash707 commented Mar 30, 2025

Towards #11719

The following examples of ChartArea will be converted to TypeScript:

  • Basic with right aligned legend
  • Labels with bottom aligned legend
  • Embedded legend
  • Embedded HTML

@patternfly-build
Copy link
Contributor

patternfly-build commented Mar 30, 2025

@Mash707 Mash707 mentioned this pull request Mar 30, 2025
18 tasks
@Mash707 Mash707 force-pushed the convert-chart-plot-to-typescript branch from 013c96d to 9b98542 Compare April 9, 2025 10:43
/* eslint-disable-next-line */
import chart_color_orange_300 from '@patternfly/react-tokens/dist/esm/chart_color_orange_300';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This may just depend on my comment in #11727 (comment), but otherwise this PR looks good

@nicolethoen nicolethoen self-requested a review April 21, 2025 14:37
@Mash707 Mash707 force-pushed the convert-chart-plot-to-typescript branch from 9b98542 to ecefffb Compare April 29, 2025 16:37
@Mash707 Mash707 requested a review from thatblindgeye April 29, 2025 16:46
@nicolethoen nicolethoen merged commit ecf9ba7 into patternfly:main May 15, 2025
13 checks passed
@patternfly-build
Copy link
Contributor

Your changes have been released in:

Thanks for your contribution! 🎉

@Mash707 Mash707 deleted the convert-chart-plot-to-typescript branch May 15, 2025 16:33
mattnolting pushed a commit to mattnolting/patternfly-react that referenced this pull request Jun 4, 2025
* chore(chart box plot) - convert to typescript

* added example - Labels with bottom aligned legend

* addded examples - embedded legend and embedded html

* disabled camel case rule

* disabled camel case rule

* disable eslint camelcase

* renamed files and functions

* remove eslint camelcase
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.

4 participants