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

Option too choose font size based on DPI scaling/Monitor #18432

Closed
SivanagBalla opened this issue Jan 15, 2025 · 1 comment
Closed

Option too choose font size based on DPI scaling/Monitor #18432

SivanagBalla opened this issue Jan 15, 2025 · 1 comment
Labels
Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. Needs-Tag-Fix Doesn't match tag requirements Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting

Comments

@SivanagBalla
Copy link

Description of the new feature

I have two displays - one is laptop 2800x1800 resolution @ 200% scaled, another is monitor 1920x1080 resolution @ 100%scaling. I often move the terminal between these two. Terminal maintains almost same character height and size but I have two problems

Problem #1 - even though character sizes are almost same, the spacing is differnet - look below image. As as result its slightly odd

Image
Image

Problem #2 - This is more personal preference kind of thing. My monitor is slightly far then the laptop monitor. so i would prefer slightly bigger font (the next size) - that makes it easier to read

So,far. I use ctrl+ ctrl- everytime I move the terminal which is becoming a bit of annoyance. Can you please include option to explicitly set font size based on DPI/Monitor. I don't mind if count of rows/columns are not same

Proposed technical implementation details

No response

@SivanagBalla SivanagBalla added the Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. label Jan 15, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Tag-Fix Doesn't match tag requirements Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels Jan 15, 2025
@carlos-zamora
Copy link
Member

Thanks for filing! We've discussed this as a team and we probably won't implement per-monitor/-dpi font specification.

However, we do have a recommendation. You can make this modification to your settings.json file and it should help with the whitespace issue. You'll have to reduce the 20px to something reasonable and you should use fractional values though.

{
    "profiles":
    {
        "defaults":
        {
            "antialiasingMode": "cleartype",
            "font":
            {
                "cellWidth": "20px"
            }
        }
    }
}

@carlos-zamora carlos-zamora closed this as not planned Won't fix, can't repro, duplicate, stale Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. Needs-Tag-Fix Doesn't match tag requirements Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting
Projects
None yet
Development

No branches or pull requests

2 participants