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

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#563 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
72/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
html, javascript
Bereich
frontend

Rechercherichtung

Beginne mit der entscheidenden Schutzbedingung in capsules/assistant/browser/agent-stream.js und der Matrix für echte Eingaben in scripts/fixtures/assistant-follow-input.html. Reproduziere PageUp und Home, während der Fokus in einem Textfeld liegt, und prüfe anschließend die Fälle mit Nachrichtenschaltflächen in Chromium und WebKit. Die Aufgabe ist abgeschlossen, wenn Tasten in Textfeldern den Nachrichtenverlauf nicht scrollen und die beiden Matrix-Erwartungen mit dem gemeldeten Verhalten übereinstimmen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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.

Vorherrschende Sprache
Rust
Sterne
6
Forks
2
Ø Merge
1 T. 3 Std.
Gemergte PRs (30 T.)
42

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus elastos/elastos-runtime

Alle Issues in elastos/elastos-runtime

Ähnliche Issues

Weitere Issues zu Rust

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.