Skip to content

Fix: LineBreak within optional chaining#335

Closed
AndreasHoffmann2 wants to merge 3 commits into
trullock:masterfrom
froebel:fixOptionalChaining
Closed

Fix: LineBreak within optional chaining#335
AndreasHoffmann2 wants to merge 3 commits into
trullock:masterfrom
froebel:fixOptionalChaining

Conversation

@AndreasHoffmann2
Copy link
Copy Markdown

We got an error from the minified code in the browser:

MicrosoftTeams-image

The linebreak between "?" and "." should not be there.

The code attached should do the trick I guess.

@trullock
Copy link
Copy Markdown
Owner

Looks right, will merge asap

@AndreasHoffmann2
Copy link
Copy Markdown
Author

Thank you!

@AndreasHoffmann2
Copy link
Copy Markdown
Author

Since we hit another very similar issue, I created a unit-test for this. And the fix above does not fix the issue.
I'll close this for the moment.

@AndreasHoffmann2
Copy link
Copy Markdown
Author

We created a unit test and did now really fix the issue.

@trullock
Copy link
Copy Markdown
Owner

Looks good, will merge asap

@AndreasHoffmann2
Copy link
Copy Markdown
Author

We found several other issues with the line-breaks and are currently working on a generic test and the necessary fixes. Please do not merge this one for the moment.

@AndreasHoffmann2
Copy link
Copy Markdown
Author

Closed since we have a better test now.

@AndreasHoffmann2
Copy link
Copy Markdown
Author

#338

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants