Skip to content

Improved new-lines handling for markdown text with background color#11530

Merged
lukasmasuch merged 2 commits intodevelopfrom
papercut-improved-background-color
Jun 6, 2025
Merged

Improved new-lines handling for markdown text with background color#11530
lukasmasuch merged 2 commits intodevelopfrom
papercut-improved-background-color

Conversation

@lukasmasuch
Copy link
Copy Markdown
Collaborator

Describe your changes

Papercut to add better multi-line wrapping in markdown with text that has a background color.

Before:
image

After:

image

Testing Plan

  • Updated snapshots

Contribution License Agreement

By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

@lukasmasuch lukasmasuch requested a review from Copilot June 5, 2025 13:37
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 5, 2025

✅ PR preview is ready!

Name Link
📦 Wheel file https://core-previews.s3-us-west-2.amazonaws.com/pr-11530/streamlit-1.45.1-py3-none-any.whl
🕹️ Preview app pr-11530.streamlit.app (☁️ Deploy here if not accessible)

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves the background color handling for multi-line markdown text by modifying the styling of markdown components.

  • Added the CSS property boxDecorationBreak: "clone" to ensure proper background rendering across broken lines.
Comments suppressed due to low confidence (1)

frontend/lib/src/components/shared/StreamlitMarkdown/styled-components.ts:263

  • Consider verifying cross-browser support for 'boxDecorationBreak: "clone"' and add vendor prefixes if necessary to ensure consistent behavior across different browsers.
boxDecorationBreak: "clone",

@lukasmasuch lukasmasuch added security-assessment-completed change:other PR contains other type of change impact:users PR changes affect end users labels Jun 5, 2025
@lukasmasuch lukasmasuch changed the title Improve markdown background color for new lines Improved new-lines handling for markdown text with background color Jun 5, 2025
@snyk-io
Copy link
Copy Markdown
Contributor

snyk-io bot commented Jun 5, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@lukasmasuch lukasmasuch merged commit 704001a into develop Jun 6, 2025
37 checks passed
@lukasmasuch lukasmasuch deleted the papercut-improved-background-color branch June 6, 2025 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:other PR contains other type of change impact:users PR changes affect end users

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants