Replies: 6 comments 17 replies
-
|
I have updated package now is not happening current version -> cogentcore.org/core v0.3.5 |
Beta Was this translation helpful? Give feedback.
-
|
it is happening again. |
Beta Was this translation helpful? Give feedback.
-
|
Thank you for reporting this. Do you have a standalone reproducible example for this issue? I can't reproduce this with a normal text editor. Also, does it happen 100% of the time, or only sometimes? |
Beta Was this translation helpful? Give feedback.
-
|
it happen randomly but not frequently |
Beta Was this translation helpful? Give feedback.
-
|
It hang on TextField also And it is not showing any crash message nothing. How do I provide more information about it ? |
Beta Was this translation helpful? Give feedback.
-
|
Does this issue happens with a basic text field with no additional configuration, or are your additional configuration steps necessary to reproduce the issue? In other words, can you reproduce the issue with just this code? core.NewTextField(b) |
Beta Was this translation helpful? Give feedback.
-
|
Beta Was this translation helpful? Give feedback.
-
|
Using context menu also(right click) hang the app |
Beta Was this translation helpful? Give feedback.
-
|
Okay, I will run this in the race detector and see if I can reproduce it. |
Beta Was this translation helpful? Give feedback.
-
|
Attaching full application -> It is a simple application store notes about anything in plain text you can add/delete/edit/search the notes it uses sqlite database. |
Beta Was this translation helpful? Give feedback.
-
|
these are the setting of TextEditor -> |
Beta Was this translation helpful? Give feedback.
-
|
i have run the app with go run -race . (v,zip) output -> |
Beta Was this translation helpful? Give feedback.
-
|
If you can see whether the app also hangs if you just have a basic text editor or text field app, that would be helpful. In other words, does an app with just this code hang when you paste as well? func main() {
b := core.NewBody()
texteditor.NewEditor(b)
b.RunMainWindow()
} |
Beta Was this translation helpful? Give feedback.
-
|
Yes it is ->
|
Beta Was this translation helpful? Give feedback.
-
|
If you want to check you can remote access my computer also. |
Beta Was this translation helpful? Give feedback.
-
|
Thank you for providing that information and screenshot. It seems like this issue has to do with your Linux operating system, where we have had issues with copy and paste before. If you could please provide the distro (Debian, Ubuntu, etc) and windowing system (X11 or Wayland) that you are using, that will help us debug this issue on our Linux device. |
Beta Was this translation helpful? Give feedback.
-
|
Beta Was this translation helpful? Give feedback.
-
|
@heejit @kkoreilly i tried to run following example with package main
import (
"cogentcore.org/core/core"
"cogentcore.org/core/events"
"cogentcore.org/core/styles"
"cogentcore.org/core/styles/abilities"
"cogentcore.org/core/texteditor"
)
func main() {
b := core.NewBody()
btn := core.NewButton(b).SetText("Message")
btn.OnClick(func(e events.Event) {
core.MessageDialog(btn, "Hello, World!", "Message")
})
vi := texteditor.NewEditor(b)
vi.Styler(func(s *styles.Style) {
s.Grow.Set(1, 1)
s.SetTextWrap(false)
s.SetAbilities(false, abilities.Hoverable)
})
b.RunMainWindow()
}and what i found:
A one of them: diff --git a/system/driver/base/app.go b/system/driver/base/app.go
index 05a6bd280..964d25267 100644
--- a/system/driver/base/app.go
+++ b/system/driver/base/app.go
@@ -29,7 +29,7 @@ type App struct {
This system.App `display:"-"`
// Mu is the main mutex protecting access to app operations, including [App.RunOnMain] functions.
- Mu sync.Mutex `display:"-"`
+ Mu sync.RWMutex `display:"-"`
// MainQueue is the queue of functions to call on the main loop.
// To add to it, use [App.RunOnMain].
diff --git a/system/driver/base/app_multi.go b/system/driver/base/app_multi.go
index 913631894..0839129b3 100644
--- a/system/driver/base/app_multi.go
+++ b/system/driver/base/app_multi.go
@@ -43,6 +43,8 @@ func NewAppMulti[W system.Window]() AppMulti[W] {
}
func (a *AppMulti[W]) NScreens() int {
+ a.Mu.RLock()
+ defer a.Mu.RUnlock()
return len(a.Screens)
}@heejit @kkoreilly Go reported 20+ races when i tried to type several lines of text and save them. so who knows what happens on Linux in display server at that time, but there is a lot of room for improvements in the core. |
Beta Was this translation helpful? Give feedback.
-
|
@AnyCPU thanks |
Beta Was this translation helpful? Give feedback.
-
this is how i create a texteditor and event handler on body for keychord ->
Beta Was this translation helpful? Give feedback.
All reactions