fix(designer): html editor styles not showing in v3

[AfroJoe759]

• Please check if the PR fulfills these requirements

• The commit message follows our guidelines
• Tests for the changes have been added (for bug fixes/features)

• What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
  Bug fix

• What is the current behavior? (You can also link to an open issue here)

• Current behavior is that for the new designer, the HTML editor adds
  tags anyplace where there is new lines which results in malformed outputs (spaces where there shouldn't be) and malformed tags resulting in styles defined in a tag to not render. The malformed HTML also appears when a user reopens a flow and checks the HTML editor.

• What is the new behavior (if this is a feature change)?

• 
  tags aren't replacing newline (\n) characters anymore, which fixes malformed HTML outputs and allows styles that aren't defined inline to be applied, which matches the bahavior of the older designer.

• Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
  N/A

• Please Include Screenshots or Videos of the intended change:
  The bug itself contains a more detailed example and explanation of the above, but here's a simple example of the difference before and after my change.

Link to bug: https://msazure.visualstudio.com/One/_workitems/edit/27066057/

Before Change Screenshot

After Change Screenshot

[hartra344]

@Eric-B-Wu can you review this one?

[Eric-B-Wu]

@Eric-B-Wu can you review this one?

yeah im just confirming on our end this doesn't break anything I think in general we wanted to keep anything in the html editor strictly in HTML-form

[Eric-B-Wu]

From what I'm seeing this is only happening in the conversion between html editor and raw html editor. I'm not seeing any consequences for us - however, because @ek68794998 was the one who worked on the raw html editor, he would know better if there are side effects. I'm going to approve so this doesn't block, but would like input from the other Eric once he's back

[Eric-B-Wu]

@hartra344 do you know why these localization strings were deleted? they seem to be still in use; however, Joe mentioned that this was added on commit/push

[hartra344]

I think there's something up with them getting pushed when generated on commit

[AfroJoe759]

@hartra344 @Eric-B-Wu do you want me to revert that specific file to what it is in the main branch?

[ek68794998]

Sorry, having trouble adding comments with the correct GitHub account. Hopefully this works. The reason this was added was to make sure that if someone entered a newline in the raw HTML editor, that it would be retained when 1) switching back to the WYSIWYG editor, 2) when saving the flow contents, and 3) if there were multiple newlines in a row. Let’s make sure we’ve validated these cases, and check with Kishor who can hopefully share some past newline bugs with some test cases we can run this through.

…

Message ID: ***@***.***>