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

[TAS-250] SnackBarの共通化 #142

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Conversation

namori999
Copy link
Collaborator

@namori999 namori999 commented Jan 25, 2025

関連のタスク issue

https://flutteruniv.slack.com/archives/C06CBUPRQ1M/p1737787130238969

対応したこと

  • SuccessSnackBarをCommonSnackBarに命名変更
     - Actionボタンを追加できるように
  • 標準のSnackBarを使っていたところをCommonSnackBarに統一

未解決事項

  • なし

実際の挙動

動作サンプル

チェックリスト

  • 実装完了後、問題が起こっていないか実際に動作確認したか
  • 補足があった方が良い/重点的にレビューが欲しい箇所にGitHub上でコメントをしたか

その他コメント

@namori999
Copy link
Collaborator Author

多分 #141 とコンフリクト起きるので、どちらかがマージされたら解消対応します!

@namori999 namori999 self-assigned this Jan 25, 2025
@namori999 namori999 force-pushed the feature/snackbar_unification branch from dc52c37 to ccdc823 Compare January 25, 2025 07:00
@schwarzwald0906 schwarzwald0906 changed the title SnackBarの統一 [TAS-250] SnackBarの統一 Jan 26, 2025
Copy link

SnackBarの共通化

@schwarzwald0906 schwarzwald0906 changed the title [TAS-250] SnackBarの統一 [TAS-250] SnackBarの共通化 Jan 26, 2025
Copy link
Collaborator

@schwarzwald0906 schwarzwald0906 left a comment

Choose a reason for hiding this comment

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

@namori999
命名に関して、
#141 でconfirm_dialogと名付けて、こちらがcommon_snack_barと名付けているのは、
dialogのパターンが複数ありうるのに対して、スナックバーのパターンが一つしかありえないからでしょうか?

もしそうなら問題ないです。

@namori999
Copy link
Collaborator Author

@namori999

命名に関して、

#141 でconfirm_dialogと名付けて、こちらがcommon_snack_barと名付けているのは、

dialogのパターンが複数ありうるのに対して、スナックバーのパターンが一つしかありえないからでしょうか?

もしそうなら問題ないです。

ConfirmDialogは、「ok」「キャンセル」などの2つのボタンがあるダイアログのことを指していて、たんにDialogというとポップアップ式で出てくるあらゆる形式のダイアログを指すという定義をしていました。
(もともとの命名がConfirmDialogだったのでそのまま使っています)

ただ、Confirm以外のダイアログであれば機能をもとにユニークな名前をつけるでしょうし(「店舗選択ダイアログ」など)、
このプロジェクト内でのCommonDialogはこれだ!という意味でCommmonDialogと命名してもいいかもしれないです。

@masa-tokyo
Copy link
Collaborator

このプロジェクト内でのCommonDialogはこれだ!という意味でCommmonDialogと命名してもいいかもしれないです。

確か元々はYes/Noを確認する、という特定の目的を持ったダイアログという意味で ConfirmDialog としていました!
ただ、その後にNoが無いパターンが加わったりと、現状踏まえると Commonで良いかなと思います。

但し、他にも CustomElevatedButtonのように Customという命名になっていたりもするので、どれかに統一させるというのは別PRに切り分けての対応で良いかなと思います!
案:

  • CustomXXX: 公式をカスタムするよ、的なニュアンス
  • CommonXXX: アプリ内で統一だよ
  • AppXXX: アプリのXXXだよ

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.

3 participants