Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"undeclared field: 'onFrame'" for X11 Window. #85

Closed
janEntikan opened this issue Apr 16, 2022 · 1 comment
Closed

"undeclared field: 'onFrame'" for X11 Window. #85

janEntikan opened this issue Apr 16, 2022 · 1 comment

Comments

@janEntikan
Copy link

In src/windy/platforms/x11.nim the Window is missing the onFrame field. Here's a diff of my fix:

--- a/src/windy/platforms/linux/x11.nim
+++ b/src/windy/platforms/linux/x11.nim
@@ -5,6 +5,7 @@ type
 
   Window* = ref object
     onCloseRequest*: Callback
+    onFrame*: Callback
     onMove*: Callback
     onResize*: Callback
     onFocusChange*: Callback
@@ -782,6 +783,8 @@ proc pollEvents(window: Window) =
         window.prevSize = ev.configure.size
         if window.onResize != nil:
           window.onResize()
+        if window.onFrame != nil:
+          window.onFrame()
 
     of xeMotion:
       window.mousePrevPos = window.mousePos
@guzba guzba mentioned this issue Apr 20, 2022
Merged
@guzba
Copy link
Collaborator

guzba commented Apr 27, 2022

This should be fixed now with this merged: #86

@guzba guzba closed this as completed Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants