Skip to content

Ensure Plotly area chart is reactive #7761

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.

Sign up for GitHub

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

Jump to bottom
Merged
akshayka merged 1 commit into from
Jan 11, 2026
Merged

Ensure Plotly area chart is reactive #7761

akshayka merged 1 commit into from
Jan 11, 2026

Conversation

@nojaf
Copy link
Contributor

@nojaf nojaf commented Jan 9, 2026

📝 Summary

This was already working, but I've added some tests to make it explicit that Plotly area charts are reactive.

I have also rearranged some tests and remove some duplicate ones after a bad merge in #7735.

🔍 Description of Changes

image

📋 Checklist

  • I have read the contributor guidelines.
  • For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
  • I have added tests for the changes made.
  • I have run the code and verified that it works as expected.
@nojaf nojaf requested a review from akshayka as a code owner January 9, 2026 10:11
@vercel
Copy link

vercel bot commented Jan 9, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
marimo-docs Ready Ready Preview, Comment Jan 9, 2026 10:11am
@akshayka akshayka changed the title Ensure Plotly area chat is reactive Jan 9, 2026
@akshayka akshayka added the enhancement New feature or request label Jan 11, 2026
akshayka
akshayka approved these changes Jan 11, 2026
View reviewed changes
Copy link
Contributor

@akshayka akshayka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @nojaf for all your improvements to our Plotly support!
@akshayka akshayka merged commit 9c8181e into marimo-team:main Jan 11, 2026
43 of 49 checks passed
@github-actions
Copy link

github-actions bot commented Jan 11, 2026

🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.19.3-dev3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

3 participants

@nojaf @akshayka @mscolnick