createWindow with visible=false does not create a hidden window about revery HOT 3 OPEN

@bryphe I suspect the solution is to expose the SDL_WINDOWFLAGS to the Sdl2.Window.create call in Window.re

I'm happy to take this on, although I expect I'll need some help since there's some C plumbing I'm not immediately familiar with :)

from revery.

Thanks @bryphe!

My time contributing to OSS is very sporadic but let's see if I can get to this :) I'll drop a comment if I start digging into this.

If anyone else is reading this and is missing the functionality feel free to jump on this 🤝

from revery.

Hi @Ragnar-H ,

Thanks for logging the issue and all the details! This would certainly be a nice improvement - it can help reduce flicker when starting up apps to start them hidden, and then show them once everything has been loaded / setup.

The simplest fix would probably be to add a flag here:

These places would also need to be updated:

• revery/packages/reason-sdl2/src/sdl2_wrapper.cpp

  Line 1667 in 9ec44ff

  argv[5]

• revery/packages/reason-sdl2/src/sdl2.re

  Line 231 in 9ec44ff

  external create:

(and then, validating that the window can be shown after being hidden is important - I think the current SDL_ShowWindow is the right thing for that, though).

A larger, but more flexible, fix would be as you suggested - expose the SDL_WINDOWFLAGS as something we can build and pass from Reason to the C stubs. We have the flags hard-coded here:

One idea would be to pass a uint to our CreateWindow API, and have a module like WindowFlags that can be used with that API, like:

module WindowFlags: {
   // Internally, this would be a uint
   type t;
   
   type flags = 
   | Hidden
   | OpenGL
   | Fullscreen
   | ...;
   
   // The default set of flags we use currently - OpenGL, HighDPI, Resizable
   let default: t;

   let make: list(flags) => t;
}
...

module Window: {
   let create: (~flags=WindowFlags.default, ...);
}

Hope that helps give some ideas!

from revery.