Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Message viewer doesn't understand Linux-style new lines #1770

Closed
montoner0 opened this issue Jul 3, 2023 · 5 comments
Closed

Message viewer doesn't understand Linux-style new lines #1770

montoner0 opened this issue Jul 3, 2023 · 5 comments

Comments

@montoner0
Copy link

Describe the bug

When a log message contains Linux-style new lines (\n instead of \r\n) they aren't rendered as new lines in the message viewers.

ℹ Computer information

  • OS: Windows 10 Version 22H2 64 bit
  • Analogy Version: 4.14 .NET framework 4.8
  • Data Provider used: Serilog

📝 Provide detailed reproduction steps (if any)

  1. Open 1.clef from the attached zip in Serilog Parser and click a message
  2. The message is shown as in the screenshot 1 below (no distinctive new lines)
  3. Open 2.clef from the attached zip in Serilog Parser and click a message
  4. The message is shown as in the screenshot 2 below (new lines are clearly present)

✔️ Expected result

1.clef and 2.clef are rendered the same way

❌ Actual result

1.clef and 2.clef are rendered differently

📷 Screenshots

image1
image2

logz.zip

@LiorBanai
Copy link
Member

Interesting.
it seems even Devexpress has no solution beside replacing the text.
https://supportcenter.devexpress.com/ticket/details/t1153819/re-newline-character-in-memoedit

I'll check options to handle it.

Thanks for reporting this.

@LiorBanai
Copy link
Member

I'm considering adding toggle in the UI which will enable the user to switch between styles. Always doing it, or checkeding, when not needed may impact performance

@montoner0
Copy link
Author

It's not ideal but still better than nothing.
What's interesting that tooltips treat the new lines properly.
image

@LiorBanai
Copy link
Member

I'll check how Devexpress implemented the tool tip. Maybe it will give idea how to do it automatically.

@LiorBanai
Copy link
Member

@montoner0
I have implemented a custom parser. Should be released in the coming days :)

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

No branches or pull requests

2 participants