Table visualization does not render \r\n as newline

[lziosi]

Found using Superset 1.3.2.
I have a text column called description in Postgresql with strings like: "\r\n\r\nTitle".
When I display these strings in a Table chart, the carriage return an line feed are not rendered (all text appears on the same line).
I can workaround this by creating a derived column with the function:

regexp_replace(description, E'[\\n\\r]+', '</br>', 'g' )

I have found an old closed defect that should have fixed this:
e762978#diff-e4cefda577d63cfaafae%5B%E2%80%A6%5D46b45c7fce61e56362e207f915
But the referenced code files (Table.css and Table.js) are not in the latest version of the repository.
This recent change:
66adcc9
shows that: "white-space: nowrap;" was changed to "white-space: normal;"
That would explain why \r\n are no longer honored (it used to be "white-space: pre;" at the time of the earlier fix).

It seems to me that "pre-wrap" might be the most useful option: https://developer.mozilla.org/en-US/docs/Web/CSS/white-space
Would it be possible to change this or introduce an option?

[yousoph]

@prosdev0107 could you take a look? thanks!

[prosdev0107]

@yousoph I carefully took an enough check and test.
\r\n didn't honor even with "white-space: pre" or "white-space: pre-wrap".
I believe that we can't control them by css whitespace property and can only control them by reg expression.

Please let me know your idea.

[yousoph]

Hey @lziosi , it looks like this functionality wasn't supported before the most recent change in 66adcc9

Do you have a SHA of if/when this worked?

[Hokwang]

I have a same problem,
I actually use superset for tables.
table should support newline.

For now, web shows newline to just one space, when I download csv file and load it to google sheet,
it shows newline.

So now I always download and use google sheet for seeing newline.

[Hokwang]

I found some link,
https://stackoverflow.com/questions/10937218/how-to-show-multiline-text-in-a-table-cell
#2118

[Hokwang]

@yousoph Could you check this issue again? I hope someone updated this issue.

[Hokwang]

@prosdev0107 Could you check this issue once again? Is there any solution?

[gaful]

I have the same issue, we come up with transforming it into <br>, but <br> appears when exporting data so it is not a really good solution.

[rusackas]

Closing this out for now, since:

1. the thread has gone pretty stale
2. I think this is a feature request more than a bug.
3. There are valid workarounds, as posted on the thread.

As always, we'd be more than happy to review a PR. And if anyone else disagrees, and thinks this is truly a bug, I'm more than happy to re-open this.