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

Navigatorの整理 #14

Merged
merged 4 commits into from
Jul 11, 2020
Merged

Navigatorの整理 #14

merged 4 commits into from
Jul 11, 2020

Conversation

thoth000
Copy link
Collaborator

issue

#12

やったこと

  • lib/main.dartにページルートを定義した。
  • 今まで使われてきたNavigator.pushNavigator.pushNamedに修正した。
  • Navigator.pushNamedでargumentsとしてデータを渡すようにした。

profile_edit.dartにも変更を加えた

  • TextTextFieldとし、名前の入力をできるようにした。
  • ProfileImageFlatButtonを親としてつけ、関数を実行できるようにした。

Copy link

@ho2ri2s ho2ri2s left a comment

Choose a reason for hiding this comment

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

基本良さそうなのでapproveです!
修正はおまかせします!

MaterialPageRoute(
builder: (context) => RoomPage(room.name),
),
'/roomPage',
Copy link

Choose a reason for hiding this comment

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

ここ、ハードコーディングだとタイポする可能性あるかなーと思ったんですが、FlutterではConstとかで定義するんですかね?
優先順位的にそんなになので対応はおまかせします!

@@ -76,7 +76,8 @@ class _ProfilePage extends StatelessWidget {
),
label: Text("編集する"),
onPressed: () {
Navigator.pushNamed<void>(context, "/profileEditPage");
Navigator.pushNamed<void>(context, "/profileEditPage",
arguments: "test");
Copy link

Choose a reason for hiding this comment

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

細かいですけど、�lib/ui/molecules/talk/list_tile.dart とインデント合わせるために "test", にしたさあります(笑)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

間違いないです笑
修正します!

Copy link

Choose a reason for hiding this comment

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

ここらへんもlinter rule 入れたいですね〜時間があればやります〜

@thoth000
Copy link
Collaborator Author

thoth000 commented Jul 11, 2020

インデントの部分のみ修正してpushしました!
チェックするのは大丈夫そうなのでMergeします!

@thoth000 thoth000 merged commit ca64ce4 into master Jul 11, 2020
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.

2 participants