Skip to content

Conversation

@Light2Dark
Copy link
Contributor

📝 Summary

adds a dismissable error banner to show that the notebook must be saved.

Screen.Recording.2025-12-11.at.10.11.28.PM.mov

🔍 Description of Changes

📋 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.

@Light2Dark Light2Dark requested a review from manzt as a code owner December 11, 2025 14:12
@vercel
Copy link

vercel bot commented Dec 11, 2025

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

Project Deployment Preview Comments Updated (UTC)
marimo-docs Ready Ready Preview Comment Dec 11, 2025 2:22pm

@Light2Dark Light2Dark changed the title add error banner add error banner for acp to save the notebook Dec 11, 2025
@mscolnick mscolnick merged commit d545d64 into main Dec 11, 2025
26 checks passed
@mscolnick mscolnick deleted the sham/fix-get-cwd-to-show-error branch December 11, 2025 18:53
@Light2Dark Light2Dark added the bug Something isn't working label Dec 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants