actualbudget/actual

[Bug]: Mobile: Enter after inserting transaction split amount > UI moves to next split but keyboard focus stays on the 1st amount

Open

#3.933 geöffnet am 5. Dez. 2024

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (7.129 Stars) (603 Forks)batch import
buggood first issueresponsiveuser interface

Beschreibung

Verified issue does not already exist?

  • I have searched and found no existing issue
  • I will be providing steps how to reproduce the bug (in most cases this will also mean uploading a demo budget file)

What happened?

I was expecting that the focus would've moved to the next split items amount if the UI moved there. Or actually, that it would've closed the keyboard

Steps to repro:

  1. be on mobile (view)
  2. Click on new transaction > enter an amount
  3. Click 'split' > type the amount > press enter (4. move to the second split, enter amount, press enter > nothing happens)

Happens on edge and on v24.11.0, Chrome 131 on android and Windows 11

https://github.com/user-attachments/assets/afa4dc33-92f6-4ca8-a3f6-50aab3739711

Where are you hosting Actual?

Pikapods

What browsers are you seeing the problem on?

Chrome

Operating System

Mobile Device

Contributor Guide

Tech Stack
javascriptreacthtmlcss
Domain
mobilefrontend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
React hooks basicsDOM event handling
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
55
Research-Richtung
Reproduce the bug in mobile view. Locate the split transaction amount input component, likely in the transaction entry form. Inspect the keydown event handler for Enter key to see why focus does not move to the next split input. Check if there is a focus management function or if the event propagation is being stopped. The UI moves to the next split, indicating state changes, but focus is not updated. A fix might involve using refs to programmatically focus the next input.