Window size changes multiple times per second and other problems under Wayland about bracket-lib

[vks]

This does fix the issue for me!

[thebracket]

I'll see if I can get a Linux VM going to test this. I'm writing on Windows, which tends to either work or not work spectacularly! I mostly do server-side Linux work, so it may take me a bit to get caught up on this one.
Thanks for letting me know!

[vks]

I also have this problem on Linux (without Wayland). It seems to be related to window decorations. They appear later than the rest of the window, and the jumps correspond to the size of the window decorations.

[thebracket]

Still trying to get a VM up and running (I'm really low on disk space!), but I suspect that the issue is in rltk.rs, specifically the lines:

WindowEvent::Resized(logical_size) => {
                    let dpi_factor = wc.window().hidpi_factor();
                    wc.resize(logical_size.to_physical(dpi_factor));
                }

So every time the console receives an update command, it tries to adjust the physical size to handle the logical size (this was from some example code). Commenting it out on Windows doesn't change anything (I think it's an attempt to work around Mac's odd resolution handling), but would mean that the "resize" event sent by the Linux system to handle the addition of window decorations wouldn't trigger a resize (which in turn seems to trigger another resize).

So I'll commit a version that comments this out - any chance one of you could update and give it a go for me, please? I'll keep trying on the VM...

[thebracket]

I'll wait to make sure I haven't caused problems anywhere before I close it, but hopefully that'll do it.

[mgedmin]

I'll open a separate bug for my input issues: #7.

[vks]

I also have the input issues you describe. They persist after the fix for the flickering.

…

On Mon, Sep 16, 2019, 19:41 Marius Gedminas ***@***.***> wrote: I'll open a separate bug for my input issues. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#6>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAIFNFZEBIYDXZXWJD2CTLQJ7AMHANCNFSM4IWQSOZA> .

[mgedmin]

This seems to be fixed for me with the latest rltk_rs.

[thebracket]

Appears to be fixed.