How to control the initial window size?

[bluenote10]

I cannot quite figure out if setting the initial window size is currently a missing feature, or if I just fail to figure it out.

One thing I tried is to control the window size via the Dynamic that can be passed to the window. Something like this:

let inner_size = Dynamic::new(Size::new(UPx::new(1000), UPx::new(800)));

build_some_widget()
    .into_window()
    .inner_size(inner_size)
    .run();

But it looks like this does not respect the initial value (which may be a bit problematic for map_each/for_each listeners that observe the size value, and actually get passed a "false" value).

Setting the widget to a fixed size also doesn't quite result in the desired behavior, because then the window cannot be resized at all, i.e., it is not just affecting the initial size. I also noticed a slight visual artifact of fixing the widget size like that: Initially, for a very short moment, the window is actually drawn in the default 800x600 size and only shortly after it gets resized to the fixed size. The size jump looks a bit weird.

Another related use case would be to control whether the window gets initialized into "maximized" state.

[ecton]

Sorry for not noticing this for so long! I let my Github notices pile up with a couple of hectic weeks, and I somehow missed this in the list.

This is a little bit of a pain point of me trying to embrace both dynamics and WindowAttributes. Initially, Cushy only had WindowAttributes and slowly more and more data is being added as dynamically accessible values. Initial window state is currently controlled via WIndowAttributes, and the dynamic is then updated with whatever winit reports via its various events.

This means that for now, setting the window's attributes' inner_size is the way to set the initial size of the window. I'll address this in the future, as doing what you've done should work, and I really should just get rid of more of the properties off of WindowAttributes.

[ecton]

I've addressed this specific attribute with this change: 60c7ef8

Since WIndowAttributes::inner_size is an option, we can fill it in when it is None with the value contained in the dynamic. I'm leaving the issue I created open so that I can still handle this better in the long-term.