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

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#888 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
75/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
csharp
Lĩnh vực
desktop-dev

Hướng nghiên cứu

Tìm kiểu AdaptiveProgressBar và mã nối thuộc tính Value của nó thành XML toast. Tái hiện với một vùng văn hóa như de-DE và giá trị 0.5, sau đó thêm hoặc cập nhật một kiểm thử hồi quy xác nhận rằng value đã nối dùng . làm dấu phân cách thập phân. Hoàn thành khi XML thanh tiến trình chứa value="0.5" dưới một vùng văn hóa dùng dấu phẩy làm dấu phân cách thập phân và các kiểm thử nối toast hiện có vẫn vượt qua.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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.

Ngôn ngữ chính
C#
Star
1.1k
Fork
168
Merge trung bình
2 giờ 48 phút
Pull request đã merge (30 ngày)
2

Chuẩn bị môi trường

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của CommunityToolkit/Windows

Tất cả issue của CommunityToolkit/Windows

Issue tương tự

Thêm issue về C#

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.