Skip to content
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

[charts] Move tooltipGetter to seriesConfig #16331

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

JCQuintas
Copy link
Member

Allow different series types to provide their own legendGetter config

@JCQuintas JCQuintas added enhancement This is not a bug, nor a new feature component: charts This is the name of the generic UI component, not the React module! labels Jan 25, 2025
@JCQuintas JCQuintas self-assigned this Jan 25, 2025
@mui-bot
Copy link

mui-bot commented Jan 25, 2025

Deploy preview: https://deploy-preview-16331--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 8565058

Copy link

codspeed-hq bot commented Jan 25, 2025

CodSpeed Performance Report

Merging #16331 will not alter performance

Comparing JCQuintas:item-tooltip-getter (8565058) with master (22f4ca7)

Summary

✅ 6 untouched benchmarks

@JCQuintas JCQuintas changed the title [charts] Move tooltipGetter to plugin system [charts] Move tooltipGetter to seriesConfig Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: charts This is the name of the generic UI component, not the React module! enhancement This is not a bug, nor a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants