Bug - LogViewer - Doesn't maintain scroll position when wrap text prop changes
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 45/100
- Issue-Typ
- Bug
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- typescript
- Bereich
- frontend
Rechercherichtung
Start with the LogViewer implementation and reproduce the issue in the PatternFly demo at the linked complex-toolbar URL: scroll to a later line, then toggle Wrap Text. Trace the rerender and confirm that the fix preserves the same log location after wrapping changes, then verify the reproduction no longer returns to line 1.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Go to this demo: https://www.patternfly.org/v4/extensions/log-viewer#with-complex-toolbar
- Scroll to a line, say line 100
- Toggle the value of the
Wrap Textcheckbox - 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.
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?
Jira Issue: PF-2009
- Vorherrschende Sprache
- TypeScript
- Sterne
- 34
- Forks
- 19
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus patternfly/react-log-viewer
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 67/100
patternfly/react-log-viewer#106 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
patternfly/react-log-viewer#101 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 25/100
patternfly/react-log-viewer#98 · 1 Kommentar ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 25/100
patternfly/react-log-viewer#95 · 2 Kommentare ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
patternfly/react-log-viewer#91 · 1 Kommentar ·
Alle Issues in patternfly/react-log-viewer
Ähnliche Issues
-
documentation
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
inu-appcenter/memorIN-frontend#106 ·
Maintainer antworten meist innerhalb von 1 Tag
-
kind/bug
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100
Maintainer antworten meist innerhalb von 7 Tagen
-
[Bug] @deck.gl/arcgis dist import resolves to unpublished @deck.gl/core source path (9.3.11, 9.4.0)Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
CSCfi/sd-search-ui#145 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Add: Cbeebies pl SDOffencheck:passed streams:add
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
Maintainer antworten meist innerhalb von 1 Tag