Skip to content

Limit series for scatter chart#3769

Merged
precious-onyenaucheya-ons merged 2 commits intomainfrom
feature/limit-colors-for-scatter-chart
Jan 5, 2026
Merged

Limit series for scatter chart#3769
precious-onyenaucheya-ons merged 2 commits intomainfrom
feature/limit-colors-for-scatter-chart

Conversation

@precious-onyenaucheya-ons
Copy link
Copy Markdown
Contributor

@precious-onyenaucheya-ons precious-onyenaucheya-ons commented Dec 18, 2025

What is the context of this PR?

Addresses ONSDESYS-734

This PR introduces a limit on the number of data series for the scatter chart to 4.

Any additional series beyond these limits will be ignored and not displayed.

How to review this PR

Open the scatter chart example .
Add more than 4 data series — confirm that only the first 4 are rendered.
Check that the component documentation clearly explains these changes.
Optionally, adjust the series count (e.g., from 3 → 5) to verify that the truncation works correctly for both themes.

Checklist

This needs to be completed by the person raising the PR.

  • I have selected the correct Assignee
  • I have linked the correct Issue

@netlify
Copy link
Copy Markdown

netlify bot commented Dec 18, 2025

Deploy Preview for ons-design-system-preview ready!

Name Link
🔨 Latest commit ce7384c
🔍 Latest deploy log https://app.netlify.com/projects/ons-design-system-preview/deploys/695b7fc6681ca500083e5a59
😎 Deploy Preview https://deploy-preview-3769--ons-design-system-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@precious-onyenaucheya-ons precious-onyenaucheya-ons marked this pull request as ready for review December 18, 2025 09:52
@precious-onyenaucheya-ons precious-onyenaucheya-ons requested a review from a team as a code owner December 18, 2025 09:52
@precious-onyenaucheya-ons precious-onyenaucheya-ons added the Enhancement Change of existing feature label Dec 18, 2025
@precious-onyenaucheya-ons precious-onyenaucheya-ons changed the title update to limit series for scatter chart Limit series for scatter chart Dec 18, 2025
@precious-onyenaucheya-ons precious-onyenaucheya-ons merged commit fae6414 into main Jan 5, 2026
14 checks passed
@precious-onyenaucheya-ons precious-onyenaucheya-ons deleted the feature/limit-colors-for-scatter-chart branch January 5, 2026 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement Change of existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants