Covid-19Radar / Covid19Radar

Open Source / i18n / iOS Android Cross Platform Contact Tracing App by exposure notification framework Xamarin App and Server Side Code
https://github.com/Covid-19Radar/Covid19Radar/blob/master/CONTRIBUTING.md
Mozilla Public License 2.0
47 stars 9 forks source link

#648対応 backキーによる画面遷移でホーム画面を経由するよう変更 #704

Closed takapi-Jr closed 4 years ago

takapi-Jr commented 4 years ago

Purpose

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

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

How to Test

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

What to Check

Verify that the following are valid

Other Information

takapi-Jr commented 4 years ago

上記を実装した場合、メニュー画面にて下記不具合がありました。

  1. メニュー画面で任意の画面を選択して遷移
  2. backボタンでホーム画面に戻る
  3. メニュー画面を開く
  4. 1.で選択した箇所に選択色がついたまま

ホーム画面に戻った場合は、メニュー画面の選択色もホーム画面に戻した方が無難です。

takapi-Jr commented 4 years ago

この実装では不具合があるので、一旦このプルリクエストをクローズします。 今後メニュー画面の選択色がない実装に変更となれば、こちらの実装で問題ないと思います。