fix: replace newline(\n) with br for text widgets.

[techbhavin]

Description

TextWidget text render imporve(`/n' issue fixed)

Fixes #7497

Type of change

• Bugfix (non-breaking change which fixes an issue)

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

Test coverage results 🧪

🟢 Total coverage has increased

// Code coverage diff between base branch:release and head branch: fix/text-wid-remove-newline 

Status	File	% Stmts	% Branch	% Funcs	% Lines
🟢	total	54.82 (0)	36.83 (0.01)	33.89 (0)	55.41 (0)
🔴	app/client/src/widgets/TextWidget/component/index.tsx	100 (0)	77.27 (-1.06)	100 (0)	100 (0)

[somangshu]

@techbhavin looks like the core problem is not the text widget, Can you confirm if this is reproducible on release env, I can seems to do that

also in the text widget \n seems to work fine

[marks0351]

@techbhavin @somangshu what's the proposed fix here?
if we are going to allow break tags shouldn't we allow any html tag that might come from a rick text editor which includes bold, italics, etc?

[somangshu]

@marks0351 The fix was actually for the new line character we see in the standup app, I thought the root cause would be the RTE or list widget, But was not able to reproduce this. I am not sure of the above change.

[somangshu]

An issue in the original package here says that

Currently it's impossible to match a string with line breaks because convertLineBreaks replaces all of them with <br/> tags which causes the matcher to get get a splitted string.

So how does the above resolve anything?

[techbhavin]

@marks0351 @somangshu

Interweave package used underneath Text widget shows new line char "\n" as it is. So we need to replace "\n" with
before forwarding the text to the lib.

Interweave handles HTML / Markdown text perfectly.

[YogeshJayaseelan]

@techbhavin @RakshaKShetty Verified this PR now \n and quotes are not displayed.

[mohanarpit]

@techbhavin @somangshu Is this PR still required? If not, can we close this PR?

I'm asking because I've closed the linked Github issue as non-reproducible.

[somangshu]

@mohanarpit we were just waiting to see if the issue crops up again, This has resolved for now, Closing this now