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

SymbolIcon broken in non english languages. #8385

Closed
ZH-PurpleFox opened this issue Apr 12, 2023 · 9 comments
Closed

SymbolIcon broken in non english languages. #8385

ZH-PurpleFox opened this issue Apr 12, 2023 · 9 comments
Labels
bug Something isn't working closed-Fixed Described behavior has been fixed. needs-triage Issue needs to be triaged by the area owners team-Controls Issue for the Controls team
Milestone

Comments

@ZH-PurpleFox
Copy link

ZH-PurpleFox commented Apr 12, 2023

Describe the bug

The control symbolIcon of Winui3 will cause garbled code in non English environments

Steps to reproduce the bug

 ##<SymbolIcon Symbol="Refresh"></SymbolIcon>

Expected behavior

The control will be garbled instead of the expected Symbol.

Screenshots

批注 2023-04-12 093249

NuGet package version

None

Windows version

Windows 10 (21H2): Build 19044

Additional context

No response

@ZH-PurpleFox ZH-PurpleFox added the bug Something isn't working label Apr 12, 2023
@Jay-o-Way
Copy link
Contributor

I know exactly why: MicrosoftDocs/winrt-api#2363
Shame on Microsoft, for not having this fixed years ago.

@Jay-o-Way
Copy link
Contributor

SymbolIcon is practically deprecated. See #5887 (comment)
Use a FontIcon.

@bpulliam bpulliam added the team-Controls Issue for the Controls team label Aug 22, 2023
@ranjeshj ranjeshj changed the title SYmbolIcon controls Unable to display correctly Bug title SymbolIcon broken in non english languages. Dec 13, 2023
@llongley llongley added the closed-Fixed Described behavior has been fixed. label Jan 18, 2024
@Jay-o-Way
Copy link
Contributor

Hi @llongley can you explain how this issue is fixed?

@microsoft-github-policy-service microsoft-github-policy-service bot added the needs-triage Issue needs to be triaged by the area owners label Jan 18, 2024
@codendone
Copy link
Contributor

@Jay-o-Way Looking at the internal issue, it looks like this was the fix: #8379. This fix was also in WinAppSDK 1.4.

@Jay-o-Way

This comment has been minimized.

@karkarl
Copy link
Contributor

karkarl commented Jan 23, 2024

Hey @Jay-o-Way, the fix for this issue was in 2 parts:

  1. Update FontIcons in existing controls for WinUI2 & WInUI3 to the appropriate glyphs
  2. Update the SymbolEnum to also map to the appropriate glyphs

The linked comment above (#8379) was to update the controls in WinUI2. The same fix was ported over to WinUI3, and a separate PR was merged in for SymbolEnum. Both these PRs are internal and serviced for WASDK 1.4. Please let me know if you are still able to repro this issue.

Please reference the comment here explaining.

@Jay-o-Way
Copy link
Contributor

Okay, so it's internal and thus invisible, right? That's good to know. A comment like such would have been nice.

@karkarl
Copy link
Contributor

karkarl commented Jan 24, 2024

Okay, so it's internal and thus invisible, right? That's good to know. A comment like such would have been nice.

Actually, WinUI3 source is available now! Here's the code that maps the glyphs for your reference: https://github.com/microsoft/microsoft-ui-xaml/blob/winui3/release/1.5-stable/dxaml/xcp/core/core/elements/icon.cpp#L448

Note that the decision to write a map rather than replacing the glyphs explicitly is to avoid any breaking changes in the SymbolEnum API (and a whole formal API review).

@Jay-o-Way
Copy link
Contributor

Jay-o-Way commented Jan 24, 2024

@karkarl thank you so much! I absolutely love the quality of your communication! You're the best!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working closed-Fixed Described behavior has been fixed. needs-triage Issue needs to be triaged by the area owners team-Controls Issue for the Controls team
Projects
None yet
Development

No branches or pull requests

6 participants