FontNameEditor rendering artifact - *just* the blue box

[RussKie]

Problem description:

FontNameEditor font preview is showing less characters.
For example on the below screenshots:

Font	.NET 4.7.2	.NET Core 3.0
Arial Black	ab	a
Forte	abc	ab
Lucida Handwriting	ab	a

Actual behavior:

.NET 4.7.2	.NET Core 3.0

🤔 ink It is possible the change may related to the change of the default font in #656.

Expected behavior:

We show the same amount of characters as with .NET FX.

It looks like we can achieve the same render by reducing the fontSize in DrawFontSample:

        private static void DrawFontSample(PaintValueEventArgs e, FontFamily fontFamily, FontStyle fontStyle)
        {
            float fontSize = (float)(e.Bounds.Height / 1.2);
            using (var font = new Font(fontFamily, fontSize, fontStyle, GraphicsUnit.Pixel))
            {
                e.Graphics.DrawString("abcd", font, SystemBrushes.ActiveCaptionText, e.Bounds);
            }
        }

Minimal repro:

• Run WinformsControlsTest in .NET 4.7.2 and .NET 3
• Compare

[RussKie]

@KlausLoeffelmann / @dreddy-work - what do you think?

[KlausLoeffelmann]

A wild guess: The font spacing in the classic framework dialog in general is different and reminds me of the old GDI+ rendering. Are we using GDI in core, and GDI+ in classic framework, so are the UseCompatibleTextRendering settings different at one point? I, btw, would prefer Core rendering, since it seems crisper, even if it changes layouts bounderies slightly. There will be many changes necessary due to HighDPI requirement adjustments, anyway, so I'd vote for let the breaking change happen. @merriemcgaw? @Tanya-Solyanik, what do you think?

Let's postpone the work assignment for this for now.

[RussKie]

Do we want to reduce the fontSize value to display more characters in the bluish box?

[RussKie]

I'm assigning to get people's attention. Once questions answered, we can reassign.

[KlausLoeffelmann]

Calling them out in posts will probably suffice. 😄

[KlausLoeffelmann]

Do we want to reduce the fontSize value to display more characters in the bluish box?

Not sure if that already had Accessibility impact, so I'd rather said no.

[KlausLoeffelmann]

Oh, I just saw you meant just the box. Well, that would work for me!

[RussKie]

😄 Yes, just the blue box.