Skip to content
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

Set scale_factor=2 only if both Retina and 5K #96

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

JoeyLemur
Copy link

On older Macs that have a 2880 x 1800 Retina display, setting scale_factor=2 makes the IDE far too large. This minor logic change will only set scale to x2 if it detects both Retina and 5K.

On older Macs that have a 2880 x 1800 Retina display, setting scale_factor=2 makes the IDE far too large.  This minor logic change will only set scale to x2 if it detects both Retina and 5K.
@FellippeHeitor
Copy link
Contributor

Is the issue happening in Catalina? Also, making the change will exclude cases where only one of the indicators appears.

@JoeyLemur
Copy link
Author

Yes, this is on my Catalina system. My thought is that it shouldn't scale by x2 unless it finds 'Retina' and '5K' in the 'system_profiler SPDisplaysDataType' output, and it does fix the scaling issue on my MBP 15" (mid-2015).

I probably should've done this before I submitted the PR, but when I get some time today, I'll throw Catalina onto a drive and see what happens on my 2109 MBP 16" -- its already on Big Sur, and doesn't have scaling problems, either with or without this PR.

@familygw
Copy link
Contributor

familygw commented Jan 8, 2021

Yes, this is on my Catalina system. My thought is that it shouldn't scale by x2 unless it finds 'Retina' and '5K' in the 'system_profiler SPDisplaysDataType' output, and it does fix the scaling issue on my MBP 15" (mid-2015).

I probably should've done this before I submitted the PR, but when I get some time today, I'll throw Catalina onto a drive and see what happens on my 2109 MBP 16" -- its already on Big Sur, and doesn't have scaling problems, either with or without this PR.

Hi!
Can you share your “ system_profiler SPDisplaysDataType”?
Retina appears for system with integrated displays, and 5K when you use external UHD/4K monitors (for example on MacPro or MacMini).

Also share the output for “uname -a” from your Catalina’s system.

Maybe a good approach will be to identify the resolution on “retina” displays.
Thanks!!

@flukiluke
Copy link
Contributor

Is this still relevant? Do we need more data?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants