mirror of
https://github.com/rivo/tview.git
synced 2026-04-26 21:35:54 +03:00
[GH-ISSUE #1020] Mouse click on InputField sets Focus to inner TextArea #738
Labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/tview#738
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @IlorDash on GitHub (Sep 1, 2024).
Original GitHub issue: https://github.com/rivo/tview/issues/1020
When using InputField, I've encountered an issue where clicking on the field with the mouse sets the focus to the inner TextArea rather than the InputField itself or its Box. This behavior prevents SetFocusFunc method from being triggered as expected.
Below is the code I am using to create an InputField. For brevity, the header and dialogue setup have been omitted::
It seems like issue arises because InputField's MouseHandler calls the TextArea's mouse handler (https://github.com/rivo/tview/blob/master/inputfield.go#L670), resulting in the TextArea gaining focus. To resolve this, I think we need to call to Box.MouseHandler() instead to ensure the InputField.Box.focus is properly triggered on mouse click.
@IlorDash commented on GitHub (Sep 8, 2024):
I discovered a bug when calling Box.MouseHandler() instead of textArea.MouseHandler(). The issue occurs when you first click to focus on an InputField, and then try to focus on a different InputField. The first field remains focused because the textArea's Box retains the focus, resulting in any typing still happening in the first InputField. Therefore, this solution doesn't work as intended. Do you have any suggestions on how to resolve this issue?
@rivo commented on GitHub (Sep 21, 2024):
The latest commit should fix the issue with the focus callback.
Regarding your second issue, can you please post a brief code snippet that I can run to reproduce it?
@IlorDash commented on GitHub (Sep 30, 2024):
Thanks, your commit really solved my problem!
To reproduce the second issue, you can use this example code:
You also need to make some changes to the
tviewpackage:i.textArea.MouseHandler()withi.Box.MouseHandler()in tview/inputfiled.go:675.To be clear, here is the
tviewversion from mygo.modwith which I reproduced the issue:v0.0.0-20240921122403-a64fc48d7654. I made my changes totview, described above, to this version.To reproduce the issue, follow these steps:
firstInputFieldwith the mouse;secondInputFieldwith the mouse;firstInputFieldinstead ofsecondInputField, because the focus remains on thefirstInputField'sBox.@rivo commented on GitHub (Nov 3, 2024):
This problem doesn't happen for me in the current version, at least not with the steps you listed. So unless you can demonstrate that it still exists in the latest version, I will consider this issue as "resolved".