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

Font ramp update #4862

Merged
merged 3 commits into from
Apr 20, 2021
Merged

Font ramp update #4862

merged 3 commits into from
Apr 20, 2021

Conversation

marksfoster
Copy link
Contributor

@marksfoster marksfoster commented Apr 20, 2021

This PR fixes a regression in BaseTextBlockStyle which causes TextBlocks in legacy code (with style BaseTextBlockStyle) to not be SemiBold.

STYLE IN GENERIC
<Style x:Key="BaseTextBlockStyle" TargetType="TextBlock"> <Setter Property="FontFamily" Value="XamlAutoFontFamily" /> <Setter Property="FontWeight" Value="SemiBold" /> <Setter Property="FontSize" Value="14" /> <Setter Property="TextTrimming" Value="None" /> <Setter Property="TextWrapping" Value="Wrap" /> <Setter Property="LineStackingStrategy" Value="MaxHeight" /> <Setter Property="TextLineBounds" Value="Full" /> </Style>

Before
image

After
image

@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Apr 20, 2021
@marksfoster
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@marksfoster
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@beervoley beervoley added the area-UIDesign UI Design, styling label Apr 20, 2021
@marksfoster marksfoster merged commit 7620b69 into master Apr 20, 2021
@marksfoster marksfoster deleted the user/markfo/textstyle branch April 20, 2021 21:05
@marksfoster marksfoster removed the needs-triage Issue needs to be triaged by the area owners label Apr 20, 2021
ranjeshj pushed a commit that referenced this pull request Apr 22, 2021
@mdtauk
Copy link
Contributor

mdtauk commented May 14, 2021

@marksfoster Do these changes take into account the change of font from Segoe UI to Segoe UI Variable?

I can only assume there is some new logic going on under the hood to adjust the OpticalSize axis in line with the font size being used - but I wonder if that will only be supported with Windows 10 21H2 onward, or if the font will be bundled with WinUI/Reunion, so all WinUI apps will benefit from the new and more consistent typographic styles?

Kinnara added a commit to Kinnara/ModernWpf that referenced this pull request Dec 14, 2023
Kinnara added a commit to Kinnara/ModernWpf that referenced this pull request Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-UIDesign UI Design, styling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants