join newlines with spaces in Html.__format__

[enricozb]

📝 Summary

Fixes a bug in Html.__format__ when the provided html text has newlines inside an html tag.

🔍 Description of Changes

Previously, newlines were effectively removed when formatting an Html object. This breaks in cases where html tags are split across multiple lines. For example,

Html("""
  <div
    style="..."
  ></div>
""")

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

📜 Reviewers

@akshayka OR @mscolnick

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
marimo-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 22, 2025 9:30pm
marimo-storybook	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 22, 2025 9:30pm

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[enricozb]

I have read the CLA Document and I hereby sign the CLA

[enricozb]

recheck

[mscolnick]

thank you! ill follow up with some tests

[mscolnick]

actually, i'm not sure if this will create other issues. cc @akshayka who has more context on this

[mscolnick]

This is definitely a bug, and your fix does fix this. Example notebook here

But i wonder if we should just not be stripping newlines, because it could also lose info, such as new lines help display something. Maybe a better solution is that we don't override __format__

[mscolnick]

i added a few more test cases to show off the current behaviour

[enricozb]

i would personally strip before splitting rather than stripping after joining

[mscolnick]

👍 will do

[github-actions]

🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.11.9-dev6