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

Bug - LogViewer - Doesn't maintain scroll position when wrap text prop changes #6

Open
snigdha920 opened this issue Dec 25, 2022 · 0 comments
Labels
bug Something isn't working

Comments

@snigdha920
Copy link

Describe the problem
A clear and concise description of the problem. Which components are affected?

I'm using the log viewer component. When I scroll to a particular line, say line 100 > then toggle the value of wrap text > the component re-renders and I'm taken to line 1.

You can see this behavior in the demo here: https://www.patternfly.org/v4/extensions/log-viewer#with-complex-toolbar

How do you reproduce the problem?
Provide steps to reproduce. A codepen or codesandbox demonstrating the problem is appreciated.

  1. Go to this demo: https://www.patternfly.org/v4/extensions/log-viewer#with-complex-toolbar
  2. Scroll to a line, say line 100
  3. Toggle the value of the Wrap Text checkbox
  4. You are taken to line 1

Expected behavior
A clear and concise description of the expected behavior.

Even after toggling the value of wrap text, the scroll position should be the same.

This is important because if the user arrives at a log line much after line 1, say something like 1000, tries to read the log by wrapping the text, and is taken to the 1st line, it will be very cumbersome for the user to find / arrive to the same line again in case they don't remember the line number.

Is this issue blocking you?
List the workaround if there is one.

I couldn't find a workaround.

Screenshots
If applicable, add screenshots to help explain the issue.

demo.mp4

What is your environment?

  • OS: Mac OS
  • Browser: Firefox

What is your product and what release date are you targeting?
I'm not sure I'm at liberty to disclose this information.

Any other information?

@snigdha920 snigdha920 added the bug Something isn't working label Dec 25, 2022
@jessiehuff jessiehuff transferred this issue from patternfly/patternfly Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Backlog
Development

No branches or pull requests

2 participants