利用状況表示のUIが直感的でない #534 の実装

[muak]

Purpose

#534 利用状況表示のUIが直感的でない
の実装です。

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

• Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

• Test the code

What to Check

設定ページで「接触の検出」または「通知」をOFFにしたときにホームページで設定を促す表示に変わり
両方の設定をONにしたときに設定が有効であることを知らせる表示に動的に変化することを確認しました。

Other Information

この表現方法に関してはまだ議論が必要かも知れませんが、実装例としてPRしました。
また言語は、日本語と英語を設定していますが、英語は仮なので、ちゃんとした翻訳が必要かと思います。

[Niccari]

こちらでも動作確認したところ、#534 で表示されている通り表示されることを確認しました。このレイアウトですと、アプリの設定がOKかわかりやすくて良いですね！

なお、HomePage.xamlについてそのままビルドしようとして場合以下のエラーによりビルドできなかったため、一旦文字列を直打ちしてテストしました(MultilingualResourcesにHomePageDescription6, HomePageDescription7を追加する必要がありそうです)。

Views/HomePage/HomePage.xaml(81,24): XamlC error XFC0101: x:Static: unable to find a public -- or accessible internal -- static field, static property, const or enum value named "HomePageDescription6" in "resources:AppResources".

** スクリーンショット(iPhone 11 Emulator) **

• 通知ON 、Bluetooth ON
  

• 通知、BluetoothのいずれかをOFF
  

[muak]

@Niccari
ご確認ありがとうございます。
MultilingualResourcesのjaとenに該当キーを追加しました。
あとエラーに関してはAppResourcesの自動生成ファイル分を含めていなかったので、これも念のため含めておきました。
これでエラーは出なくなると思います。

[Niccari]

@muak ご対応ありがとうございます。コミット 19f62aa についてビルドできること確認いたしました(コミット a2f9f36 の場合、一部キーの重複などによりエラーとなりました)。

ちなみに本変更に付随して、HomePageViewModelにあるIsAvailableの判定条件はUserDataModel側に定義して一律化できればより保守しやすくなると思います。

[muak]

@Niccari
a2f9f36 はDesigner.csでコンフリクトしていました。

確かに、UserDataModelにReadOnly Propertyで定義しておくと良いですね。
ついでにここでやっておきますか。
ちょっとローカルのgitがおかしくなってしまったので別のタイミングで対応したいと思います🙇🏻‍♂️

[muak]

コンフリクト解消をgithub上で行ってしまったため、現masterの変更を取り入れてしまったので
それをrevertしてややこしいことになってしまいました…
もし取り入れる場合はちゃんとしたの作り直します。

[kazumihirose]

こちらは厚生労働省によって運用されているCOCOAのコードではありません。
大変恐縮なのですが、厚生労働省(COCOA)に再度Issue/PRなど頂ければ幸いです。
何卒宜しくお願い致します。
ただ、非常にありがたいご提案でして、Covid19Radarにもマージを検討したいと思います。