Skip to content

feat(data-viz): add simple chart type support #4318

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 63 commits into
base: main
Choose a base branch
from

Conversation

krisantrobus
Copy link
Collaborator

Data Visualization Enhancements:

  • Updated Highcharts native tooltip styles in the @twilio-paste/data-visualization-library to align with Paste styles, including font family adjustments.
  • Added helper functions and types in the @twilio-paste/data-visualization-library to support creating Highcharts options for supported chart types.
  • Extended the ChartProvider component to allow usage of options for supported chart types.

Updates

  • Updated the ChartProvider documentation for supported charts and enhanced API section /components/chart-provider/api
  • Added multiple examples to the /foundations/data-visualization/engineering/chart-types page

@github-actions github-actions bot added the Type: Tests Adds tests to the code label May 19, 2025
@@ -1,13 +1,13 @@
const sidebarNavigationDisclosures = [
"introduction",
"foundations",
"components",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

moved the components higher as data-visualization now exists in components and foundations so need to be open to test both

@@ -20,6 +20,7 @@
"scripts": {
"build": "yarn clean && NODE_ENV=production node build.js && tsc",
"build:js": "NODE_ENV=development node build.js",
"build:typedocs": "tsx ../../../tools/build/generate-type-docs --skipPropsNameRule",
Copy link
Collaborator

Choose a reason for hiding this comment

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

👏

Copy link
Collaborator

@nkrantz nkrantz left a comment

Choose a reason for hiding this comment

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

great tests and examples!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label May 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Components Related to the component library (core) of this system Area: Doc Site Related to the documentation website Area: Infrastructure Related to our infrastructure and build tools Area: Repo lgtm This PR has been approved by a maintainer size:XXL This PR changes 1000+ lines, ignoring generated files. Type: Documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants