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

#78 画像詳細ビューをphotoViewパッケージに変更 #101

Merged
merged 2 commits into from
Aug 2, 2023

Conversation

RikitoNoto
Copy link
Collaborator

@RikitoNoto RikitoNoto commented Aug 2, 2023

Issue

close #78

説明

画像詳細ビューで最大まで拡大後、縮小ができないバグの修正。
InteractiveViewerを使用して画像詳細画面を表示していたが、ジェスチャーの反応範囲は画像のサイズが変わっても変わらないため、縮小ができないように感じてしまう。
ジェスチャーの範囲が画面全体で作成されているphoto_viewを使用して当画面を作成しなおした。

UI

mottai_issue78

その他

photo_viewは画面全体を覆うウィジェットのため、以前までつけていた画像外をタップすると画面が閉じる処理は削除しています。

チェックリスト

  • PR の冒頭に関連する Issue 番号を記載しましたか?
  • 本 PR の変更に関して、エディタや IDE で意図しない警告は増えていませんか?(lint 警告やタイポなど)
  • Issue の完了の定義は満たせていますか?
  • 当該 Issue のスレッドで、レビュワーにレビュー依頼をしましたか?

@codecov
Copy link

codecov bot commented Aug 2, 2023

Codecov Report

Patch coverage has no change and project coverage change: +1.27% 🎉

Comparison is base (188c625) 20.29% compared to head (2bd5c69) 21.56%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #101      +/-   ##
==========================================
+ Coverage   20.29%   21.56%   +1.27%     
==========================================
  Files          11       11              
  Lines         271      255      -16     
==========================================
  Hits           55       55              
+ Misses        216      200      -16     
Flag Coverage Δ
unittests_dart_flutter_common 11.79% <0.00%> (+0.89%) ⬆️
unittests_firebase_common ∅ <ø> (∅)
unittests_mottai_flutter_app 53.33% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
..._flutter_common/lib/src/widgets/generic_image.dart 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Owner

@kosukesaigusa kosukesaigusa left a comment

Choose a reason for hiding this comment

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

実機確認しました、完璧でした!!

@RikitoNoto RikitoNoto merged commit 3dff26f into main Aug 2, 2023
6 of 7 checks passed
@RikitoNoto RikitoNoto deleted the 78_fix_cant_scaling_image branch August 2, 2023 15:18
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.

画像詳細Viewで最大までズーム後、縮小ができない
2 participants