Skip to content

feat: Add bubble series to cartesian chart component#191

Open
avinashbot wants to merge 11 commits intomainfrom
feat-bubble-charts
Open

feat: Add bubble series to cartesian chart component#191
avinashbot wants to merge 11 commits intomainfrom
feat-bubble-charts

Conversation

@avinashbot
Copy link
Copy Markdown
Member

@avinashbot avinashbot commented Mar 24, 2026

Description

Adds a new "bubble" type for chart series, which accepts a z value in addition to x and y.

Screenshot 2026-03-24 at 02 56 06

Related links, issue #, if available: AWSUI-61535

How has this been tested?

Modified existing unit tests that cover all series types to include bubble series. There's also a new dev page for the bubble chart, which (if I understand it correctly) is now screenshot-tested.

Also reviewed it with our accessibility specialists and visual designers. And of course, manually on some local browsers with VoiceOver.

Review checklist

The following items are to be evaluated by the author(s) and the reviewer(s).

Correctness

  • Changes include appropriate documentation updates.
  • Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
  • Changes do not include unsupported browser features, see CONTRIBUTING.md.
  • Changes were manually tested for accessibility, see accessibility guidelines.

Security

Testing

  • Changes are covered with new/existing unit tests?
  • Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@avinashbot avinashbot marked this pull request as ready for review March 24, 2026 02:00
@avinashbot avinashbot requested a review from a team as a code owner March 24, 2026 02:00
@avinashbot avinashbot requested review from cansuaa and pan-kot and removed request for a team March 24, 2026 02:00
cansuaa
cansuaa previously approved these changes Mar 24, 2026
@pan-kot
Copy link
Copy Markdown
Member

pan-kot commented Mar 27, 2026

@georgylobko, let's change getPointAccessibleDescription to include bubble size - or should we track it as a follow-up change?

avinashbot and others added 10 commits March 27, 2026 17:04
* chore: Z axis for the bubble chart

* chore: small fix

* chore: Upd types and snapshots

* chore: Additional tests

* chore: Refactoring

* refactor: Changes z-axis to bubble-axis (#195)

* poc

* z -> size

* fix: tests

* small fix

* chore: Update snapshots

* chore: Update snapshots

* chore: Update snapshots

---------

Co-authored-by: Georgii Lobko <globko@amazon.de>

---------

Co-authored-by: Andrei Zhaleznichenka <zhalea@amazon.de>
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.

4 participants