Comment by ~eliasnaur on ~eliasnaur/gio
SGTM, except that I prefer a plain field:
img.Filter = paint.FilterNearest
Comment by ~eliasnaur on ~eliasnaur/gio
I don't have much bandwidth. Can you dig a little deeper, bisect, maybe get the (I assume) native error code for the error message and look it up?
Just from the error message itself, it sounds like the window message queue is overflowing. Are you calling some
app.Windowmethod on the closed window? If so, the fix may simply be a missing check forw.dead. Now that I think about it,Wakeupshould not even be available for dead windows, so there may be a race somewhere.
Comment by ~eliasnaur on ~eliasnaur/gio
REPORTED RESOLVED FIXED
Comment by ~eliasnaur on ~eliasnaur/gio
REPORTED RESOLVED FIXED
Comment by ~eliasnaur on ~eliasnaur/gio
I'd like one or more patch(es) sent through the mailing list or to GitHub. See https://gioui.org/doc/contribute for details.
on ~eliasnaur/gio
Hi ~eliasnaur, shall I submit a patch or expect it to be done from your side?
Whatever you say :)
Thanks a lot for helping me out with small steps and best practices :)
Comment by ~eliasnaur on ~eliasnaur/gio
You can put
isMobilein the case expressions:
case e.Name == key.NameUpArrow && e.Modifiers == 0 && isMobile
Comment by ~eliasnaur on ~eliasnaur/gio
Seems reasonable, except that you should move the
GOOSbefore the switch and check once.
Comment by ~eliasnaur on ~eliasnaur/gio
You could check for
runtime.GOOSequal to "android" or "ios".
Comment by ~eliasnaur on ~eliasnaur/gio
Did you change
https://git.sr.ht/~eliasnaur/gio/tree/main/item/app/window.go#L863
To use Arrow names? You probably also need to disable directional focus movement on non-mobile platforms.