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

オープン 初心者向け
  • 何か問題がありますか？
#888 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
75/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
csharp
領域
desktop-dev

調査の方向性

AdaptiveProgressBar 型と、その Value プロパティをトースト XML にシリアライズするコードを探してください。de-DE のようなカルチャーと 0.5 という値で再現し、シリアライズされた value が小数点区切りとして . を使用することを確認する回帰テストを追加または更新してください。完了の基準は、カンマを小数点区切りとして使用するカルチャーでも、プログレスバーの XML に value="0.5" が含まれ、既存のトースト シリアライゼーションテストが引き続き成功することです。

索引モデルが issue の本文から書いたものです。

説明

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.

主要言語
C#
スター
1.1k
フォーク
168
平均マージ
2時間 48分
マージ済み PR（30日）
2

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

CommunityToolkit/Windows のほかの issue

CommunityToolkit/Windows の issue をすべて見る

似ている issue

C# の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。