ピクトグラムで、「すべきこと」を上に、「すべきでないこと」を下に並べる

[yusuke]

ピクトグラムで、「すべきこと」を上に、「すべきでないこと」を下に並べる

Issue 番号 / Issue ID

• Close 接触状況を表示する画面のピクトグラムで、「すべきこと」と「すべきでないこと」が無秩序に並んでいる #186

目的 / Purpose

ピクトグラムで表現しているアクションのうちどれがすべきことで、どれがすべきでないことなのか、より明瞭にします。

破壊的変更をもたらしますか / Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request の種類 / Pull Request type

[ ] 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]
dotnet restore

コードの検証 / Test the code

確認事項 / What to check

• 正しくピクトグラムの並びが反映される

その他 / Other information

---

Internal IDs:

• PBI 2800

[yusuke]

日本語が読めなくても回避すべきことであることが分かるように改善したバージョンのPRも別途挙げさせて頂きます。
そちらの方向でよろしければこちらのPRはクローズしていただけければ。

[keiji]

ありがとうございます。
英語に関しては別途画像リソースがありますが、さまざまな人が見るという意味ではピクトグラムだけでわかったほうがいいですね。

https://github.com/cocoa-mhlw/cocoa/blob/master/Covid19Radar/Covid19Radar.Android/Resources/drawable-xxhdpi/Nocontact10.png

Android側への反映もあるので、まずはアクセシビリティの問題（ #164 ）に収容します！

[yusuke]

バッジを付けて漢字が読めなくても分かりやすくしたバージョンもリクエストさせていただきました→ #188

[yusuke]

@keiji あら、英語版のリソースがあったんですね。私の環境(iPhone SE 2020、iOS14.5.1)ですと日本語版リソースが表示されています。完全に別件になってしまいますが。

[keiji]

現象としては #77 に近いかもしれませんね。確認してIssue立てます

[yusuke]

#188 でAndroidや英語版のリソースも含め包括的に修正していますのでこちらはひとまずクローズさせて頂きます。

[yusuke]

@keiji バッジ無しでAndroid / iOS、日本語/英語のリソースを差し替えました。

[keiji]

動作確認結果
とても美しい仕上がり。

日本語

英語

iOSの英語版については #77 の影響で日本語画像が表示されている。

[yusuke]

レビューありがとうございます。画像もクッキリして、アプリサイズも減ります💕

[keiji]

@yusuke 取り込み方向で動いているこのPRですが、先に取り込まれたPull Request #218 の影響でコンフリクトが起きています。
お手数ですが対応をいただけると助かります。

#218に合わせてデフォルト言語を英語にしたいので、次の操作になります。

• Covid19Radar/Covid19Radar.iOS/Resources/ 直下にある画像を削除（ディレクトリは消さない）
• en.lproj に追加した Nocontact10*.png を Base.lproj に移動
• en.lproj には Nocontact10*.png を配置しない（Base.lprojのものが表示される）
• zh.lproj から Nocontact10*.png を削除（中国語のリソースがないため。Base.lprojのものが表示される）

操作の結果、英語のリソースがBase.lprojに、日本語のリソースがja.lprojに、次のように配置されるイメージです。

Covid19Radar/Covid19Radar.iOS/Resources/Base.lproj/Nocontact10@1x.png
Covid19Radar/Covid19Radar.iOS/Resources/Base.lproj/Nocontact10@2x.png
Covid19Radar/Covid19Radar.iOS/Resources/Base.lproj/Nocontact10@3x.png
Covid19Radar/Covid19Radar.iOS/Resources/ja.lproj/Nocontact10@1x.png
Covid19Radar/Covid19Radar.iOS/Resources/ja.lproj/Nocontact10@2x.png
Covid19Radar/Covid19Radar.iOS/Resources/ja.lproj/Nocontact10@3x.png

[yusuke]

@keiji developをマージして、以前の型式だった3つのファイルを差し替えました。ご確認の程お願いいたします。

[keiji]

LGTM!

対応ありがとうございました！

[cocoa-dev]

確認しました。問題ありませんのでApproveします。