Skip to content

Conversation

JCQuintas
Copy link
Member

Added initial definition and guidelines for each of the main chart types.

Questions

Question # 1

The current "guidelines" section, is too much "in the face" of the user. What would be the best approach?

Ideas

  • Move it to the bottom of their respective pages
  • Move it below an initial description and chart example
  • Move it to a different subpage of the main component > Bars > Definition

Question # 2

Should we provide links referring to further readings in external websites and references for the content shown?

Eg:

#### Further Reading

For more in-depth guidance on bar chart best practices and design principles:

- **[Data-to-Viz: Bar Chart](link)** - Comprehensive guide on when and how to use bar charts effectively

Question # 3

Should we use bullet points and short text (current approach).
Or long text with some sporadic bullet points, like (https://www.data-to-viz.com/graph/barplot.html)

@JCQuintas JCQuintas self-assigned this Aug 28, 2025
@JCQuintas JCQuintas added type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. scope: charts Changes related to the charts. labels Aug 28, 2025
@mui-bot
Copy link

mui-bot commented Aug 28, 2025

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

Updated pages:

Bundle size report

Bundle Parsed size Gzip size
@mui/x-data-grid 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-pro 0B(0.00%) ▼-2B(0.00%)
@mui/x-data-grid-premium 0B(0.00%) ▼-1B(0.00%)
@mui/x-charts 0B(0.00%) 0B(0.00%)
@mui/x-charts-pro 0B(0.00%) 🔺+1B(0.00%)
@mui/x-date-pickers 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers-pro 0B(0.00%) ▼-1B(0.00%)
@mui/x-tree-view 0B(0.00%) 0B(0.00%)
@mui/x-tree-view-pro 0B(0.00%) 🔺+3B(+0.01%)

Details of bundle changes

Generated by 🚫 dangerJS against ed74ca5

Copy link

codspeed-hq bot commented Aug 28, 2025

CodSpeed Performance Report

Merging #19364 will not alter performance

Comparing JCQuintas:charts-guidelines (ed74ca5) with master (3dcf6c2)1

Summary

✅ 10 untouched benchmarks

Footnotes

  1. No successful run was found on master (22f5b49) during the generation of this report, so 3dcf6c2 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@alexfauquette
Copy link
Member

Personal preferences

Q1. Move it to a different subpage of the main component > Bars > Definition
Q2. Looks like a good idea for SEO
Q3. Depends on Q1 decision. If we go with subpage we can use longer text with demo to illustrate. Otherwise, your bullet points look great for information in the page

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: charts Changes related to the charts. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants