ナビゲーションデバッグ画面にTextFieldを配置 - Githubissues

yumemi-inc / flutter-mobile-project-template

MIT License

38 stars 7 forks source link

ナビゲーションデバッグ画面にTextFieldを配置 #293

Closed mj-hd closed 4 months ago

mj-hd commented 4 months ago

概要

refer to #268
今までのナビゲーションデバッグ画面では、path paramやquery paramが存在した場合に画面遷移できませんでした
解決するために、一度TextFieldを経由して手動で編集できるようにしました

（元々はmacrosなどでうまいことGoRouteDataからサンプルパラメータを持って来れないかと試してたのですが、挫折してしまいました...手動入力の形で対応です）

レビュー観点

UIや体験が微妙かもしれません。また、他の妙案もあるかもしれません。ぜひご指摘いただけると幸いです

レビューレベル

[x] Lv1: ぱっとみて違和感がないかチェックして Approve する
[ ] Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証して Approve する
[ ] Lv3: 実際に環境で動作確認したうえで Approve する

レビュー優先度

[ ] すぐに見てもらいたい ( hotfix など ) 🚀
[ ] 今日中に見てもらいたい 🚗
[ ] 今日〜明日中で見てもらいたい 🚶
[x] 数日以内で見てもらいたい 🐢

画像 / 動画

Screenshot_1721993176

確認したこと

適当なQuery/Path Parameterが必要なルートに対して、TextFieldで書き換えた上で画面遷移できること

動作確認手順

Debug Mode画面へ遷移し、Go Navigation Debug Pageを選択
適当な選択肢を選び、TextFieldを書き換えたのちにGoボタンをおす

備考

github-actions[bot] commented 4 months ago

Ready for review :rocket:

github-actions[bot] commented 4 months ago

Visit the preview URL for this PR (updated for commit 40ee76f):

https://flutter-mobile-project-template-catalog--pr293-debug-p-efyjb4s7.web.app

_{(expires Wed, 07 Aug 2024 05:38:09 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23}

blendthink commented 4 months ago

明日レビュー予定です 🙏