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

When line is too big, it surpass the next line and mixes both end of current line and begins of next line. #3

Open
pimentelleo opened this issue Oct 28, 2022 · 8 comments · Fixed by #62
Labels
Milestone

Comments

@pimentelleo
Copy link

pimentelleo commented Oct 28, 2022

image

Workaround is set hasLineNumbers to false, this just occour when lineNumbers is enabled

@jdrews
Copy link

jdrews commented Feb 14, 2023

This is still happening when hasLineNumbers is set to false on 4.87.100.

@jdrews
Copy link

jdrews commented Mar 2, 2024

Confirmed still a problem on react-log-viewer version 5.1.0

image

@andrewballantyne
Copy link

@nicolethoen Is there any motion on this? We are running into it in OpenShift AI as well

Copy link

🎉 This issue has been resolved in version 5.2.0-prerelease.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@pimentelleo
Copy link
Author

Glad to see it, thanks! I'll test soon

@dlabaj dlabaj reopened this May 8, 2024
@dlabaj
Copy link
Collaborator

dlabaj commented May 8, 2024

Issue was still being seen on OCP when there is data coming back over stderr & stdout. Reopening for further investigations.

@dlabaj
Copy link
Collaborator

dlabaj commented May 8, 2024

Messages are being sent over stdout and stderr due to the script that is run with cockpit.spawn

The code is doing a CURL to retrieve the data.

@jdrews
Copy link

jdrews commented Jun 1, 2024

I have tried out react-log-viewer version 5.2.0 and I'm still seeing this bug.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Not started
Development

Successfully merging a pull request may close this issue.

7 participants