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 建立於 2024年12月5日

在 GitHub 查看
 (0 留言) (0 反應) (0 負責人)JavaScript (7,129 star) (603 fork)batch import
buggood first issueresponsiveuser interface

描述

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

貢獻者指南

技術棧
javascriptreacthtmlcss
領域
mobilefrontend
議題類型
bug
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
3
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-3 hours
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
fresh
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
clear
前置要求
React hooks basicsDOM event handling
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
55
研究方向
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.