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

WebViewの初回Loadingとpull-to-refreshの実装 #108

Merged
merged 18 commits into from
Feb 8, 2024

Conversation

iseruuuuu
Copy link
Contributor

@iseruuuuu iseruuuuu commented Jan 31, 2024

概要

#26 の途中までのPRとなります

レビュー観点

レビューレベル

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

レビュー優先度

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

画像 / 動画

iOS Android
Simulator.Screen.Recording.-.iPhone.15.Pro.-.2024-02-06.at.10.24.58.mp4
untitled.webm

動作確認手順

備考

@iseruuuuu iseruuuuu changed the title [WIP]WebViewの残りのタスクの実装 [WIP]WebViewの初回Loadingとpull-to-refreshの実装をしました Feb 1, 2024
@iseruuuuu iseruuuuu changed the title [WIP]WebViewの初回Loadingとpull-to-refreshの実装をしました [WIP]WebViewの初回Loadingとpull-to-refreshの実装 Feb 1, 2024
@iseruuuuu iseruuuuu changed the title [WIP]WebViewの初回Loadingとpull-to-refreshの実装 WebViewの初回Loadingとpull-to-refreshの実装 Feb 2, 2024
@iseruuuuu iseruuuuu marked this pull request as ready for review February 2, 2024 00:00
Copy link

github-actions bot commented Feb 2, 2024

Ready for review 🚀

@iseruuuuu iseruuuuu marked this pull request as draft February 6, 2024 01:19
@iseruuuuu iseruuuuu changed the title WebViewの初回Loadingとpull-to-refreshの実装 [WIP]WebViewの初回Loadingとpull-to-refreshの実装 Feb 6, 2024
@iseruuuuu iseruuuuu marked this pull request as ready for review February 6, 2024 07:48
Copy link

github-actions bot commented Feb 6, 2024

Ready for review 🚀

@iseruuuuu
Copy link
Contributor Author

@warahiko @Yamasaki-pan961
改めてですが、お手隙の際にレビューお願いします🙇

@Yamasaki-pan961
Copy link
Contributor

@iseruuuuu
PRのタイトルにWIPとついているので消しておいてください!

Copy link
Contributor

@Yamasaki-pan961 Yamasaki-pan961 left a comment

Choose a reason for hiding this comment

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

一旦コメントしましたので、ご確認お願いします。

packages/features/webview/lib/src/ui/web_view.dart Outdated Show resolved Hide resolved
packages/features/webview/lib/src/ui/web_view.dart Outdated Show resolved Hide resolved
@iseruuuuu iseruuuuu changed the title [WIP]WebViewの初回Loadingとpull-to-refreshの実装 WebViewの初回Loadingとpull-to-refreshの実装 Feb 6, 2024
Comment on lines +90 to +96
onProgressChanged: (_, __) async {
if (await _webViewController!.canGoBack()) {
canPop.value = false;
} else {
canPop.value = true;
}
},
Copy link
Contributor Author

Choose a reason for hiding this comment

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

memo-badge
次のページに遷移したときや戻ったときに毎回呼ばれるメソッドだと判断し、canPopの制御・管理の記載をしてみました

packages/features/webview/lib/src/ui/web_view.dart Outdated Show resolved Hide resolved
packages/features/webview/lib/src/ui/web_view.dart Outdated Show resolved Hide resolved
Copy link
Contributor

@Yamasaki-pan961 Yamasaki-pan961 left a comment

Choose a reason for hiding this comment

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

ご対応ありがとうございます!!

StatefulHookWidgetをStatefulWidgetまたはHookWidgetのどちらかに寄せることができそうと思いましたが、状態がhooksでコントローラがライフサイクルメソッドで管理されていてわかりやすいので、問題ないと思いました。

LGTM

@iseruuuuu
Copy link
Contributor Author

LGTMいただきありがとうございます!!!
マージさせていただきます!

@iseruuuuu iseruuuuu merged commit 404a454 into main Feb 8, 2024
3 checks passed
@iseruuuuu iseruuuuu deleted the add_webview_load_error_pulltorefresh branch February 8, 2024 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants