Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add announcement for multiple pill sheet #446

Merged
merged 13 commits into from
Dec 25, 2021

Conversation

bannzai
Copy link
Owner

@bannzai bannzai commented Dec 22, 2021

Abstract

ヤーズフレックス等のユーザー向けにピルシートグループで実現可能なことを案内する.
手段としてはNotificationBarに案内を表示してタップしたら使い方の説明のモーダルを出す

Reason
目的: ヤーズフレックス対応をちゃんとできることを伝えたい
伝わらないと行けない要素:
複数シート
服用日付表示モード
途中休薬
新規:
複数シート
初期設定で通るので伝わる。記録画面まで行けばUIで複数枚あることが理解できる可能性が高い
服用日付表示モード
初期設定にないので知らない可能性がある
デフォルトが服用日付表示じゃないので、あとから変更する必要がある
デフォルトを服用日付表示にしちゃえば良さそう。
服用日付表示がデフォルトで困ることもなさそう
途中休薬
初期設定にない。が、記録画面を見れば見つけられる可能性が高い
既存
複数シート
手動で新しくシートを作るまでは気づかないかもしれない
服用日付表示モード
記録画面の表示モードをタップしないと見つけられない・知らないままの可能性がある
途中休薬
記録画面にあるので気付く可能性が高い
課題まとめ:
新規ユーザーはデフォルトを日付表示モードにするだけで気付きそう(新規の提案)
既存ユーザーは確かに気づかない可能性がある
というわけでどちらかというと既存ユーザー向けで考えちゃえば良さそう
手段
起動後にモーダルを出す
新規・既存に問わずに伝わりそうだからそういう意味だと良い
課題感としては読まずに閉じる可能性がある
「飲んだ」時にモーダルを出す
これも新規・既存に問わずに伝わりそうだからそういう意味だと良い
ただ、「飲んだ」押す前に離脱する可能性も少なからずある
これも課題感としては読まずに閉じる可能性がある
なんか適当なタイミングでモーダルを出す
既存にのみ有効そう
適当なタイミングが完全に手探りなので正解が求めにくい割に実装も難しくなりそう
これも課題感としては読まずに閉じる可能性がある
NotificationBarに出す=>モーダルを出す
個人的にこれで良い気がした
条件によるけど、既存には伝えられる。新規にも伝えられそう(こっちはおまけ)
とはいえ条件は一番優先しちゃって良さそう。 :x: ボタン押すまで出し続ける。興味なかったら閉じるだろうし。閉じない人はどの文言でも読んでも無視しちゃうのだろう。って気持ち
特定のバージョンの間だけ出すのもあり、既存向けの訴求だと僕は思っているから
いきなり閉じられる心配も減る
良いと思った改案:
NotificationBarに出す=>モーダルを出す


showAnnouncementMultiplePillSheet(BuildContext context) async {
final sharedPreferences = await SharedPreferences.getInstance();
// if (sharedPreferences.getBool(
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • これを外す

@bannzai bannzai merged commit 7e0592c into main Dec 25, 2021
@bannzai bannzai deleted the imp/view/announcement_multiple_pillsheet branch December 25, 2021 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant