-
Notifications
You must be signed in to change notification settings - Fork 47
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
Better font rendering on OSX #353
base: master
Are you sure you want to change the base?
Conversation
even after a fresh build, doesn't seem to do anything to the fonts |
Alan 3:52 PM You can force it with a "defaults read /path/to/plist" as I recall. Another thing you can do is copy it to a new location. This has confused me many times. |
@allywarner If we can get this one to work, it should be merged before the version update, yeah? |
@dcwhite I think so! It'd be a nice addition. |
I think the problem is the actual bundle plist file is not getting this change. I'll check the cpack code |
Yes, there are deeper problems here. The plist file that winds up in the bundle is hard-coded and located in src/Resources/. So that one would get the NSHighResolutionCapable key addition. There are two .plist.in files in the CMake/Modules directory, and they're both ignored (and both are a mess of duplicated keys it seems). So this needs more clean up. I'll investigate more tomorrow. |
@jessdtate We figured out how to get improved font rendering (make the same small change, but to a different plist file) but uncovered some sloppy messiness related to the plist files in the repo. Should we (a) clean up the messiness now or (b) make the quick fix to get the fonts working before the release? |
let's fix it in the next release. I think we are ready to go otherwise. |
There should only be one plist file.
Not sure if this will make a change.
add NS principal class
…Seg3D into OSX_font_rendering
No change
Fixes widgets being squished as well on OSX.