-
Notifications
You must be signed in to change notification settings - Fork 59
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
Epileptic bug on MacOS #23
Comments
I was seeing a similar issue in Linux/X11 with an intel card. I changed the hi-dpi value in settings.json to false, and was able to avoid the issue. "hiDPI": false, |
Also got this issue on Mac OS.
|
At least on MacOS it seems that when using hiDPI the scale is reversed. I.e. in https://github.com/danfragoso/thdwb/blob/master/mustard/window.go#L35:
That makes the window content fill the window (but then it goes back to small with flicker on resize, so I think the same needs to be done wherever the window resize code is). |
I got this issue on Mac OS, too. |
@raff That seems plausible for the scale issue. The resize callback is here: https://github.com/danfragoso/thdwb/blob/master/mustard/window.go#L139 FWIW |
Hi, sorry for the delay, I've been really busy with work this couple of weeks. I fixed the scaling issue on linux with commit 088ec74, can you guys try it again? @ronangaillard @dyaskur @montyanderson @raff @bigwhite @ajzaff |
@dyaskur, if you applied https://github.com/ajzaff/thdwb/tree/feature-content-scale-23, make sure you have hiDPI=true in settings.json. The fix is still not perfect (on resizes there are cases depending on the window size where the screen goes white) but mostly works. |
I cloned your project and run
make
and I was left with this epileptic bug :Maybe this has something to do with high DPI screens ?
Commit I used :
532f88c
System info :
The text was updated successfully, but these errors were encountered: