Bug - LogViewer - Doesn't maintain scroll position when wrap text prop changes
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 45/100
- Tipo de issue
- Bug
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- typescript
- Domínio
- frontend
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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
- Linguagem predominante
- TypeScript
- Estrelas
- 34
- Forks
- 19
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de patternfly/react-log-viewer
-
Search within Log viewer can cause the parent container to shift and break the layout, when embedded in a scrollable parentTalvez já em andamento @fao89 assumiu há 152 dias. Aberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 67/100
patternfly/react-log-viewer#106 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 45/100
patternfly/react-log-viewer#101 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 25/100
patternfly/react-log-viewer#98 · 1 comentário ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 25/100
patternfly/react-log-viewer#95 · 2 comentários ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 35/100
patternfly/react-log-viewer#91 · 1 comentário ·
Todas as issues de patternfly/react-log-viewer
Issues semelhantes
-
backend refactor
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 73/100
NIAEFEUP/tts#927 · 1 comentário · 1 reação ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
callstackincubator/react-native-harness#296 ·
Mantenedores costumam responder em até 1 dia
-
status:in-linear
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 67/100
n8n-io/n8n#42293 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
verified
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 67/100
Mantenedores costumam responder em até 5 dias
-
Add inputmode attribute support (forwarded to the inner input)Talvez livre de novo Um pull request para esta issue foi fechado sem ser mesclado. Abertaenhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
Neovici/cosmoz-input#467 ·
Mantenedores costumam responder em até 1 dia