added saveWindowState / moved writeWindowConfig to onCloseRequest (#127)

* added saveWindowState / moved writeWindowConfig to onCloseRequest

issue #114

* Update src/figuro/renderer/opengl.nim

* Update src/figuro/renderer/opengl.nim

---------

Co-authored-by: Jaremy Creechley <creechley@gmail.com>

Author: derek-v-s

src/figuro/common/nodes/uinodes.nim

@@ -48,6 +48,7 @@ type
     windowStyle*: FrameStyle
     theme*: Theme
     configFile*: string
+    saveWindowState*: bool
 
   Figuro* = ref object of Agent
     frame*: WeakRef[AppFrame]

src/figuro/renderer/opengl.nim

@@ -83,10 +83,12 @@ proc configureWindowEvents(renderer: Renderer) =
 
   window.onCloseRequest = proc() =
     notice "onCloseRequest"
+    if renderer.frame[].saveWindowState: 
+      writeWindowConfig(window, winCfgFile)
     app.running = false
 
   window.onMove = proc() =
-    writeWindowConfig(window, winCfgFile)
+    discard
     # debug "window moved: ", pos= window.pos
 
   window.onResize = proc() =

src/figuro/ui/core.nim

@@ -204,7 +204,7 @@ template connectDefaults*[T](node: T) =
     when compiles(SignalTypes.tick(T)):
       connect(node, doTick, node, T.tick(), acceptVoidSlot = true)
 
-proc newAppFrame*[T](root: T, size: (UiScalar, UiScalar), style = DecoratedResizable): AppFrame =
+proc newAppFrame*[T](root: T, size: (UiScalar, UiScalar), style = DecoratedResizable, saveWindowState = true): AppFrame =
   mixin draw
   if root == nil:
     raise newException(NilAccessDefect, "must set root")
@@ -218,6 +218,7 @@ proc newAppFrame*[T](root: T, size: (UiScalar, UiScalar), style = DecoratedResiz
   frame.window.box.w = size[0].UiScalar
   frame.window.box.h = size[1].UiScalar
   frame.windowStyle = style
+  frame.saveWindowState = saveWindowState
   refresh(root)
   return frame
 

tests/tinput.nim

@@ -35,5 +35,5 @@ proc draw*(self: Main) {.slot.} =
           this.setText "hello world"
 
 var main = Main.new()
-var frame = newAppFrame(main, size=(720'ui, 140'ui))
+var frame = newAppFrame(main, size=(720'ui, 140'ui), saveWindowState = false)
 startFiguro(frame)