marimo
marimo copied to clipboard
improvement: Add "left" and "right" to `position` argument in `show_code`
📝 Summary
mo.show_code() has a position option that currently has two possible values, "above" and "below". For the Slides view it would be nice to have code and output side-by-side. This tries to add that.
🔍 Description of Changes
Basically just copied the cases for above/below and replaced the vstack with hstack. Not sure if this has implications I'm unaware of.
📋 Checklist
- [x] 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).
- [x] I have added tests for the changes made.
- [ ] I have run the code and verified that it works as expected.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| marimo-docs | 🛑 Canceled (Inspect) | Apr 1, 2025 8:30am | ||
| marimo-storybook | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Apr 1, 2025 8:30am |
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.
I have read the CLA Document and I hereby sign the CLA
Christoph Pohl seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.
I have read the CLA Document and I hereby sign the CLA
hey @Psirus - let me know if you are still interested in trying to get this merged in? we'd like to try to clean up some open PRs
I still think it would be nice to have, but I have no clue about the marimo code base, nor any web development for that matter, nor the time to take care of this. So totally understandable if you just close it, also fine by me.