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

Abierto Apto para principiantes
  • ¿Algo no está bien?
#888 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
75/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
csharp
Área
desktop-dev

Línea de trabajo

Encuentra el tipo AdaptiveProgressBar y el código que serializa su propiedad Value en XML de toast. Reproduce el problema con una cultura como de-DE y un valor de 0.5; después, añade o actualiza una prueba de regresión que compruebe que el value serializado usa . como separador decimal. La tarea está completa cuando el XML de la barra de progreso contiene value="0.5" en una cultura con coma decimal y las pruebas existentes de serialización de toast siguen aprobándose.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.

Lenguaje dominante
C#
Estrellas
1.1k
Forks
168
Merge medio
2 h 48 min
PR fusionados (30 d)
2

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de CommunityToolkit/Windows

Todos los issues de CommunityToolkit/Windows

Issues similares

Más issues de C#

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.