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.

Sign up for GitHub

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

Jump to bottom

Create CustomRefreshIndicator - edited for upgrade #246

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

Create CustomRefreshIndicator - edited for upgrade #246

wants to merge 6 commits into from

Conversation

thomaskim1130
Copy link
Contributor

@thomaskim1130 thomaskim1130 commented Nov 14, 2024 •
edited
Loading

Overview

로딩을 위해 pull down하는 거리 감소. 편의성 증진

Changes

  • refreshIndicator.adaptive()를 ScrollNotification 클래스를 활용해 custom화 함.
  • 기존 pull down 50(변경 불가) 에서 30으로 조정. (변경 가능)
  • main_page.dart, post_list_show_page.dart, user_page.dart, post_view_page 등 페이지에 적용

Implementaion Method

  • utils/refresh_indicator.dart를 추가하여 메소드 customRefreshIndicator 생성

After Changes

  • scrollDownLength를 customize할 수 있음
  • radius, color, 애니메이션 딜레이 등도 모두 customize 가능
  • refresh 함수를 호출하는 중 중복되지 않음

Related Issues

  • LoadingIndicator(화면 중앙)와 동시에 나타나는 현상이 발생하기도 함

Rollback Scenario

  1. customRefreshIndicator 대신 refreshIndicator.adaptive 사용
  2. AlwaysScrollablePhysics의 parent 변경하기

TODO

  • user_view_page.dart 에도 적용. (DONE)
  • refreshIndicator 다수 중복. Wrapper 함수 제작? (DONE)
  • DarkMode 적용

@thomaskim1130
Add(refresh_indicator.dart): created new customRefreshIndicator
c9e5409 
Created new CustomRefreshIndicator that:
- doesn't rely on any global keys
- thus doesn't cause state issues
- based on actual animation + NotificationListener
- can customize radius, pull down length, offset, etc.
- based on CupertinoActionIndicator
@thomaskim1130
Feat: applied customRefreshIndicator
87647db 
Applied customRefreshIndicator widget to :
mainPage
notificationPage
postListShowPage
postViewPage
userPage
userViewPage

(also modified physics for android)
@thomaskim1130
Fix(main_page.dart): fixed
64b0f16 
added back asynchronous operation
@thomaskim1130 thomaskim1130 added the enhancement New feature or request label Nov 14, 2024
@thomaskim1130 thomaskim1130 added this to the v1.3.0 milestone Nov 14, 2024
@thomaskim1130 thomaskim1130 self-assigned this Nov 14, 2024
@thomaskim1130 thomaskim1130 mentioned this pull request Nov 14, 2024
Closed
@thomaskim1130 thomaskim1130 linked an issue Nov 15, 2024 that may be closed by this pull request
[Feature] CustomRefreshIndicator 적용하기 #224
Open
@thomaskim1130 thomaskim1130 marked this pull request as ready for review November 21, 2024 13:03
@sangohkim sangohkim requested a review from RGLie November 21, 2024 13:38
sangohkim
sangohkim requested changes Nov 28, 2024
View reviewed changes
Copy link
Collaborator

@sangohkim sangohkim left a comment

Choose a reason for hiding this comment

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

RefreshIndicator radius 관련한 간단한 수정 요청 드렸습니다. 시뮬레이터에서 본 결과 전체적으로 잘 동작하는것 같습니다! 안드로이드에서는 CustomRefreshIndicator만 cupertino 스타일이고 나머지는 material 디자인에 맞춰져있어서 부자연스러울 수도 있겠다는 생각이 드는데 CustomRefreshIndicator를 iOS 일 때만 적용해보는건 어떠신가요?

@thomaskim1130
Feat(refresh_indicator.dart): multi platform compatibility
56d76ab 
Applied CustomRefreshIndicator ONLY for iOS (Cupertino), replaced with Material for Android.
Reset default radius to 10.0
@thomaskim1130
Merge branch 'dev' into feature/change-refresh-pulldown-edit
e2f7762
@thomaskim1130
Fix(theme_info.dart): Color bug fix
6a53f88 
scroll 시 그림자 발생하는 Bug fix, AppBarTheme을 ThemeData 내부에 도입하면서, scrolledUnderElevation=0.0으로 설정하여 해결
sangohkim
sangohkim requested changes Feb 19, 2025
View reviewed changes
Copy link
Collaborator

@sangohkim sangohkim left a comment

Choose a reason for hiding this comment

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

image

```flutter run``하고 나서 홈화면으로 들어가면 위 에러가 발생하여 수정 요청드립니다.
기종은 iphone 16 (iOS 18.1), Pixel (Android API 34) 입니다

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sangohkim sangohkim sangohkim requested changes

@sparcscasio sparcscasio Awaiting requested review from sparcscasio

@RGLie RGLie Awaiting requested review from RGLie

Requested changes must be addressed to merge this pull request.

Assignees

@thomaskim1130 thomaskim1130

Labels
enhancement New feature or request
Projects
None yet
Milestone
v1.3.0
Development

Successfully merging this pull request may close these issues.

[Feature] CustomRefreshIndicator 적용하기
2 participants
@thomaskim1130 @sangohkim