Skip to content

feat: Add Scatter to the S2 package (pre-alpha) - #869

Open
tanugarg04 wants to merge 2 commits into
mainfrom
s2-scatter
Open

feat: Add Scatter to the S2 package (pre-alpha) #869
tanugarg04 wants to merge 2 commits into
mainfrom
s2-scatter

Conversation

@tanugarg04

@tanugarg04 tanugarg04 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Add Scatter to the S2 package (pre-alpha)

Description

Ports Scatter into @spectrum-charts/react-spectrum-charts-s2 as a pre-alpha
component — full prop support, plus its child components ScatterPath,
ScatterAnnotation, Trendline, and TrendlineAnnotation. Also renames the
package's rc export tier to pre-alpha (./rc./pre-alpha) and fixes two
pre-existing bugs uncovered along the way.

The underlying vega-spec-builder-s2 spec-generation logic for all of these
already existed and needed no changes — this PR is entirely the React
component/adapter layer plus build/docs wiring.

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

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.

1 participant