AdaptiveProgressBar serializes Value attribute using system culture instead of InvariantCulture (breaks Toast progress bar on non-US locales)

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#888 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
75/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
csharp
Bereich
desktop-dev

Rechercherichtung

Finde den Typ AdaptiveProgressBar und den Code, der seine Eigenschaft Value in Toast-XML serialisiert. Reproduziere dies mit einer Kultur wie de-DE und einem Wert von 0.5, und füge dann einen Regressionstest hinzu oder aktualisiere ihn, der prüft, dass der serialisierte value als Dezimaltrennzeichen . verwendet. Fertig bedeutet, dass das XML des Fortschrittsbalkens unter einer Kultur mit Komma als Dezimaltrennzeichen value="0.5" enthält und die vorhandenen Tests zur Toast-Serialisierung weiterhin erfolgreich sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Describe the bug

When using ToastContentBuilder.AddProgressBar on a Windows machine with a regional culture that uses a comma as a decimal separator (such as Russian ru-RU, German de-DE, French fr-FR, etc.), the resulting XML for the Toast notification is generated with a comma instead of a dot.
Windows Notification Center strictly expects a dot separator (.) for fractional values in XML attributes. Because it fails to parse the comma (,), the progress bar on the screen is always rendered as completely empty (0%) for any value between 0.0 and 1.0. The only exception is exactly 1.0, which happens to work because it formats as a whole integer "1" without any fractional digits.

Environment & Conditions

• OS: Windows 10 / Windows 11
• Critical Condition: Windows Regional Format must be set to a culture that uses a comma as a decimal separator (e.g., German, Spanish, Russian).
• Framework: .NET 8 / .NET 9 / .NET 11 (Windows SDK)
• Library: CommunityToolkit.WinUI.Notifications (or Microsoft.Toolkit.Uwp.Notifications)

Steps to reproduce
1. Change the OS thread culture or Windows Regional Format to Russian (ru-RU) as sample.
2. Execute the following C# code to build a toast notification:

using CommunityToolkit.WinUI.Notifications;

var tb = new ToastContentBuilder()
    .AddText("UPS Status")
    .AddProgressBar("Battery Charge", 0.5); // Expected 50% progress

// Extract the raw XML generated by the toolkit
Windows.Data.Xml.Dom.XmlDocument xDoc = tb.GetToastContent().GetXml();
string serializedXml = xDoc.GetXml();

System.Console.WriteLine(serializedXml);

When inspecting the serializedXml string, you will see that the <progress> tag has a comma in its value:

xml: <progress value="0,5" title="Battery Charge" ... />

Because of value="0,5", Windows Notification Center drops the value to 0, and the UI shows an empty progress bar.
Expected behavior

The toolkit should always serialize the Value property inside AdaptiveProgressBar using CultureInfo.InvariantCulture, regardless of the user's current OS locale. The output should look like this:

xml: <progress value="0.5" title="Battery Charge" ... />

Screenshots

No response

Code Platform
  • UWP
  • WinAppSDK / WinUI 3
  • Web Assembly (WASM)
  • Android
  • iOS
  • MacOS
  • Linux / GTK
Windows Build Number
  • Windows 10 1809 (Build 17763)
  • Windows 10 1903 (Build 18362)
  • Windows 10 1909 (Build 18363)
  • Windows 10 2004 (Build 19041)
  • Windows 10 20H2 (Build 19042)
  • Windows 10 21H1 (Build 19043)
  • Windows 10 21H2 (Build 19044)
  • Windows 10 22H2 (Build 19045)
  • Windows 11 21H2 (Build 22000)
  • Windows 11 22H2 (Build 22621)
  • Windows 11 23H2 (Build 22631)
  • Windows 11 24H2 (Build 26100)
  • Windows 11 25H2 (Build 26200)
  • Windows 11 26H1 (Build 28000)
  • Other (specify)
Other Windows Build number

No response

App minimum and target SDK version
  • Windows 10, version 1809 (Build 17763)
  • Windows 10, version 1903 (Build 18362)
  • Windows 10, version 1909 (Build 18363)
  • Windows 10, version 2004 (Build 19041)
  • Windows 10, version 2104 (Build 20348)
  • Windows 11, version 22H2 (Build 22000)
  • Windows 11, version 24H2 (Build 26100)
  • Windows 11, version 25H2 (Build 26200)
  • Windows 11, version 26H1 (Build 28000)
  • Other (specify)
Other SDK version

No response

Visual Studio Version

No response

Visual Studio Build Number

No response

Device form factor

No response

Additional context

No response

Help us help you

Yes, I'd like to be assigned to work on this item.

Vorherrschende Sprache
C#
Sterne
1.1k
Forks
168
Ø Merge
2 Std. 48 Min.
Gemergte PRs (30 T.)
2

Entwicklungsumgebung

In Codespaces öffnen

Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus CommunityToolkit/Windows

Alle Issues in CommunityToolkit/Windows

Ähnliche Issues

Weitere Issues zu C#

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.