Any use of FindFontWithLocalizedName is broken and requires refactoring #21442

Closed
opened 2026-01-31 07:44:47 +00:00 by claunia · 2 comments
Owner

Originally created by @zadjii-msft on GitHub (Mar 26, 2024).

Originally assigned to: @lhecker on GitHub.

From #16821

Font features and axes have multiple blocking bugs right now anyway (I mean even without this PR), so I figured that we can do a follow-up refactoring later where we fix both issues: The existing bugs and the incompatibilities introduced by this one. I'm half done making the changes to fix this and they're highly related

Originally created by @zadjii-msft on GitHub (Mar 26, 2024). Originally assigned to: @lhecker on GitHub. From #16821 > Font features and axes have multiple blocking bugs right now anyway (I mean even without this PR), so I figured that we can do a follow-up refactoring later where we fix both issues: The existing bugs and the incompatibilities introduced by this one. I'm half done making the changes to fix this and they're highly related
Author
Owner

@zadjii-msft commented on GitHub (Apr 29, 2024):

@lhecker I was told this was blocking for 1.21 - are we fine shipping 1.21 with this?

@zadjii-msft commented on GitHub (Apr 29, 2024): @lhecker I was told this was blocking for 1.21 - are we fine shipping 1.21 with this?
Author
Owner

@lhecker commented on GitHub (Apr 29, 2024):

No. I'm currently working on this issue.

Technically this should be a trivial change but funneling this into XAML turns out is harder than writing the renderer lol. "Oh this doesn't have a ItemsSource you need to instantiate UI elements yourself. Oh this is only a getter, needs a manual injection. Oh you can't set Items in the Click handler or it gets unresponsive. Oh you can't have generic types in a DataTemplate, needs a custom wrapper. Oh you can't reference the VM from inside a DataTemplate. Oh…" Hilarious lol.

@lhecker commented on GitHub (Apr 29, 2024): No. I'm currently working on this issue. Technically this should be a trivial change but funneling this into XAML turns out is harder than writing the renderer lol. "Oh this doesn't have a ItemsSource you need to instantiate UI elements yourself. Oh this is only a getter, needs a manual injection. Oh you can't set Items in the Click handler or it gets unresponsive. Oh you can't have generic types in a DataTemplate, needs a custom wrapper. Oh you can't reference the VM from inside a DataTemplate. Oh…" Hilarious lol.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#21442