Cell content modal doesn't preserve newlines in multiline text

[theshazkhan]

Screenshot

Description

When clicking on a cell containing multiline text (e.g. stack traces with newline characters), the modal popup displays all the text on a single line instead of preserving the line breaks.

Expected behaviour: Newlines in the cell content should be rendered as line breaks in the modal.
Actual behaviour: All text appears on one continuous line, making stack traces and other multiline content
difficult to read.

Suggested fix: Add white-space: pre-wrap to the .ant-modal-body CSS class. This preserves newline characters and renders them correctly.

[dosubot]

Hi @theshazkhan! I'm Dosu and I’m helping the superset team.

This happens because the modal uses Ant Design's default styling, which collapses newlines, so multiline text like stack traces appears as a single line. While adding white-space: pre-wrap to .ant-modal-body would fix the issue, it would also affect all modals in Superset, potentially breaking layouts that rely on inline spacing (like forms or multi-column rows) see example.

A safer fix is to scope the CSS to only the cell content modal. You can do this by adding a custom class to the modal (e.g., cell-content-modal) and targeting it in your CSS:

.cell-content-modal .ant-modal-body {
  white-space: pre-wrap;
}

Or, if you control the modal content, wrap the multiline text in a <pre> or a styled component with white-space: pre-wrap. This preserves newlines without impacting other modals. Superset's Emotion styling guidelines recommend customizing Ant Design components with styled components for this kind of targeted override see docs.

Test the fix by opening the cell content modal with multiline text and verifying that line breaks are preserved, while other modals remain unaffected.

To reply, just mention @dosu.

---

^{How did I do? Good | Irrelevant | Incorrect | Verbose | Hallucination | Report 🐛 | Other}

   

[rusackas]

The bot sounds right! Would you want to take a crack at opening a PR?