mirror of
https://github.com/rivo/tview.git
synced 2026-04-26 21:35:54 +03:00
[GH-ISSUE #797] Window freeze if selecting textview #588
Labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/tview#588
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 @dingxiong on GitHub (Jan 22, 2023).
Original GitHub issue: https://github.com/rivo/tview/issues/797
Hey guys, I encounter a problem with textview in a form. The window freezes after I try to select a text view.
The tview version I am using is
v0.0.0-20230104153304-892d1a2eb0da. The code isI tried to select the the notes fields
This is just a demo.multiple times, then the UI freezes. By selection, I mean I put the mouse cursor on the message, and press the muse left button, and scroll to the right.Below is the debug info.
seems goroutine get stuck at acquiring a mutex. below is the stack trace of goroutine 1.
Any idea of what is happening here?
@dingxiong commented on GitHub (Jan 26, 2023):
@digitallyserviced Thanks. I can try. What is the
focusedabove?The code I have is copied from https://github.com/rivo/tview/blob/master/demos/form/main.go
Does it mean that the demo example also need to be updated?
@dingxiong commented on GitHub (Jan 27, 2023):
Hi @digitallyserviced thanks for a quick reply. Yeah. I am using OSX Iterm2, but not in tmux or ssh.
I made a recording below. Each time I selected the textview, the focus moved to the next row until it hit textview itself, it then get stuck and become unresponsive.
https://user-images.githubusercontent.com/5496402/215018170-d5b9c8c6-bb76-4005-bc9f-8ea9910acd66.mov
@dingxiong commented on GitHub (Jan 27, 2023):
Thanks for a quick fix. Sorry how to include your change? I tried to follow https://stackoverflow.com/questions/42761820/how-to-get-another-branch-instead-of-default-branch-with-go-get but seems not working.
@dingxiong commented on GitHub (Feb 8, 2023):
@digitallyserviced Thanks.
go mod edit -replaceworks for me. The patch works great!Any estimation when the patch will be merged?
@rivo commented on GitHub (Feb 8, 2023):
Text views in forms are read-only elements and thus should not be selectable. The keyboard input handler already handled this but this affected the mouse handler in the wrong way. Should be fixed now.