-
Notifications
You must be signed in to change notification settings - Fork 14
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
Resizing is wonky. (On macOS, missing frames cause scaling. On GNU/Linux, a blackout effect occurs.) #322
Comments
Uh oh. How do we think this happened @Subject38? Might you have time to test with me via TeamViewer again some time soon? |
Did I split the scale into (x,y) components and forget a |
Also if Seth has no time @eliheuer can test with me just as well. I could not find an affordable macOS virtualization service with GPU support so have not been able to determine another way to test, and the Foundation is (I am) naturally quite against the idea of purchasing an Apple computer to do local testing. |
@ctrlcctrlv It is a bit hard to see what is going on in just a screenshot, so I made a YouTube video showing the bug here: https://www.youtube.com/watch?v=VzSWypCgkdQ This is on a 2020 Apple M1 Mac Mini, macOS 11.6.5. |
I just tested on Arch Linux and was able to resize the window without any stretching of the UI, video example here: https://www.youtube.com/watch?v=ZmyjAzNq7hs So this is a MacOS bug, happy to help look into this, I'll see if I get the same thing on a older Intel non-M1 mac next. |
I have determined the fastest way to fixing Mac issues is for me to just use someone else's Mac PC and try a bunch of things until I fix it. Thanks for showing me the issue, it's just as weird as I thought. For sure @Subject38 and I would have noticed this. So, I can't really fix it until someone has time to dedicate to a remote session. Because I also tried with @Subject38 to just send patches I thought might work in the past and it was a ton of useless work for both of us, in terms of patch management, copying, pasting, it not working, me getting back a video, etc. |
I updated to macOS 12.5.1 tonight and I still get the same stretching when resizing the window. So macOS 11 was not the problem. |
Just tested on my local machine and I can confirm this issue exists. I feel like I might have noticed it before but didn't realise it was a big issue since after resizing all the UI elements go back to normal. I'm honestly not entirely sure what might be causing this behavior in the first place... |
Oh so this isn't even that big a deal but @Subject38 there's a bigger problem I'm glad I tried to do a full build again on @t0lva's PC because I literally was on the phone w/her for an hour trying to build and it kept erroring out and it turns out we're blocked on aclysma/skulpin#108 (comment) for new VulkanSDK versions What VulkanSDK do you have? |
(That's @t01va not @t0lva.) |
(Housekeeping: I'm downgrading this bug to minor and making a new one for the VulkanSDK update issue.) |
Looks like version |
I just tested this on an Intel mac on macOS 11 and still get the same stretching. I think from a UI/UX perspective it is a major issue, but I'm a UI snob I guess. |
@eliheuer I agree it looks really wonky and ugly (same with the blackout effect that happens on Linux). I'll be really honest, I just don't know how to fix it. Like there are callbacks and Vulkan something or other going on here, it's all wrapped up with Skulpin's Vulkan renderer in general and I am just as likely to make it worse as make it better. @MatthewBlanchard is usually our 3D expert but he's been a bit MIA of late so if he doesn't fix it I don't know when/if it will be, and personally I don't see it as a major problem compared to all the other ones, so that's why I've tagged it |
This issue has been merged with, and supersedes, #295. |
By the blackout effect, are y'all referring to the flickering/resetting effect that I receive? Screen recording to demonstrate the behavior on my system (Linux/NixOS/Wayland, using native Wayland, not XWayland). I don't see any scaling here. Wondering if this is somewhere upstream rather than an issue here - cross-platform WM abstractions are tricky to get right. I'll keep playing around. |
Yes, the issues are one and the same and have the same root upstream cause. |
It's worth noting what I wrote at imgui-rs/imgui-rs#582 (comment). To fix #323 we will be switching in the new year to @lucasmerlin's egui_skia, OpenGL by default, with Vulkan optionally, across all GUI MFEK modules like this one. That might fix this; in my tests so far it does. It also might not. But @jakeisnt the core GUI rendering has such large planned changes I don't think it's worth fixing this issue unless it's through the planned upgrade path. :-) |
I'm very happy to hear about egui_skia and the big changes to the GUI rendering. Thanks! |
See the screenshot below, this seems distracting and hopefully unnecessary. MacOS 11
The text was updated successfully, but these errors were encountered: