Skip to content

add base class method and override#8892

Draft
Light2Dark wants to merge 3 commits intodm/mo-5366from
sham/ui-element-approach
Draft

add base class method and override#8892
Light2Dark wants to merge 3 commits intodm/mo-5366from
sham/ui-element-approach

Conversation

@Light2Dark
Copy link
Copy Markdown
Collaborator

📝 Summary

This is another approach for #8857.
Instead of patching the subclass, we just add a method in the base class and override it.

🔍 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).
  • Tests have been added for the changes made.
  • Documentation has been updated where applicable, including docstrings for API changes.
  • Pull request title is a good summary of the changes - it will be used in the release notes.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 27, 2026

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

Project Deployment Actions Updated (UTC)
marimo-docs Ready Ready Preview, Comment Mar 27, 2026 3:14am

Request Review

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