Assistant: PageUp and Home in a text field scroll the transcript

Aberta Para iniciantes
  • Algo errado?
#563 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
72/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
html, javascript
Domínio
frontend

Direção de pesquisa

Comece pela proteção principal em capsules/assistant/browser/agent-stream.js e pela matriz de entradas reais em scripts/fixtures/assistant-follow-input.html. Reproduza PageUp e Home enquanto o foco estiver em um campo de texto e, em seguida, verifique os casos dos botões de mensagem no Chromium e no WebKit. O trabalho estará concluído quando as teclas em campos de texto não rolarem o histórico de mensagens e as duas expectativas da matriz corresponderem ao comportamento relatado.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

In Assistant, PageUp or Home pressed while typing in a text field (input, textarea, select or [contenteditable]) scrolls the whole transcript, even when the field itself has nothing to scroll. Same on develop (30b166b05) and on the Jump to latest fix branch, in Chromium and in system WebKit (Safari 26.6.2).

Measured with real key events (Grok review of fix/assistant-jump-flash at 0f7ac56f4, report /private/tmp/cursor-runs/review-jump-flash-r10.report.md, section "Shared cases to file"): Chromium PageUp leaves the transcript at top 1820 (380 px from the end), WebKit at top 1840, Home goes to top 0. The key guard in capsules/assistant/browser/agent-stream.js returns for these targets but does not stop the key, so the browser scrolls the nearest scrollable ancestor, the transcript.

Expected: keys typed inside a text field act on that field only. The transcript moves only for keys aimed at it (focus on the transcript, a message button, or the page body).

Follow behaviour after that scroll is correct and is not part of this issue: the reader really moved, so follow turns off and a growing reply does not pull them back.

When fixing, also correct two expectations in the real-input matrix (scripts/fixtures/assistant-follow-input.html) that the review found wrong: Shift+Space on a message button and WebKit ArrowUp on a message button do not scroll, so they should expect follow to stay on, not a scroll away.

Linguagem predominante
Rust
Estrelas
6
Forks
2
Merge médio
1d 3h
PRs com merge (30d)
42

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de elastos/elastos-runtime

Todas as issues de elastos/elastos-runtime

Issues semelhantes

Mais issues de Rust

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.