[Feat] #178-bottomsheet라이브러리로 구현 #181

hooni0918 · 2025-02-09T18:36:50Z

🔗 연결된 이슈

close: [feat] 바텀시트 라이브러리 교체 #178

📄 작업 내용

바텀시트 라이브러리 구현
바텀시트 중복된 코드들 제거

동작 내용은 기존과 동일해서 생략합니다.

💻 주요 코드 설명

FlexSheet 라이브러리를 추가했습니다. 스타눌러주세요
기존 BottomSheetStyle, CustomBottomSheet, FixedBottomSheetView, BottomSheetList, SearchResultBottomSheet을 삭제하고 라이브러리로 통합 관리하게 되었습니다. 궁금하시면 라이브러리 코드 뜯어보시면 이해가 될겁니다.

지금 코드는 라이브러리를 가져다 사용하는게 많아서 라이브러리 사용법으로 PR설명을 갈음하겟습니다.

FlexibleBottomSheet(
    currentStyle: $currentStyle,
    style: .defaultFlex
) {
    // 사용자 인터랙션에 따라 높이가 변하는 바텀시트
    YourContent()
}

스타일 설정

// 기본 고정 스타일
FlexSheetStyle(
    fixedHeight: UIScreen.main.bounds.height * 0.5
)

// 기본 유동 스타일
.defaultFlex

// 인터랙티브 스타일
.interactiveFlex

👀 기타 더 이야기해볼 점

라이브러리 연동관련해서 코드가 좀 많이 더럽습니다 커밋단위말고 해당 PR 만 봐주세요!
스유 정말 어렵네요,, 라이브러리도 어렵다..

참...많은일이 있었습니다 하하

초록색 부분은 가독성용

thingineeer

라이브러리로 만드셔서 보기 좋네요! 다른 것도 모듈화한다면 도움이 될 것 같습니다.

thingineeer · 2025-02-10T10:13:31Z

Spoony-iOS/Spoony-iOS/Source/Feature/SearchLocation/SearchLocation.swift

        .navigationBarHidden(true)
+        .navigationBarHidden(true)


하나 지워도 괜찮을 것 같아요!

엥? 이게뭐죠ㅋㅋㅋ

thingineeer · 2025-02-10T10:14:49Z

Spoony-iOS/Spoony-iOS/Network/Model/ResturantpickListResponse.swift

-               lhs.latitude == rhs.latitude &&
-               lhs.longitude == rhs.longitude &&
-               lhs.categoryColorResponse == rhs.categoryColorResponse
+        return lhs.placeId == rhs.placeId 


필요한 부분만 남기는거 깔끔하네요

thingineeer · 2025-02-10T10:16:17Z

.../Spoony-iOS/Resource/Components/BottomSheet/SearchResult/SearchLocationBottomSheetView.swift

+    @State private var currentStyle: BottomSheetStyle = .half
+
+    var body: some View {
+        GeometryReader { geo in


반드시 GeometryReader를 사용해야 한다면 GeometryReader도 FlexSheet에 넣어도 된다고 생각이 듭니다 .. 근데 GeometryReader를 쓰는 이유가 뭐였죠??

geo 값은 사용이 안 되는데 와일드카드 패턴을 써도 좋을 것 같아요

GeometryReader는 바텀시트의 확장조건 ( .mini .half .full) 의 상황이 비율로 움직이고 있었는데 이 비율에 대한 부분을 감지하기 위해 사용되었습니다. 이부분은 라이브러리로 구현을 해놓은 상태이구요.
현재는 바텀시트가 full 인 상황에서 스크롤뷰를 움직일때 스크롤뷰의 스크롤 영역조절을 위해 작성을 해놧었는데,, 필요가 없었네요!
현재 geo 매개변수가 사용되지 않는것도 스크롤뷰처리를 따로 하지 않아도 되기 때문에 사용자체가 될곳이 없습니다.
해당 부분 통으로 삭제하도록 할게요!

thingineeer · 2025-02-10T10:17:51Z

Spoony-iOS/Spoony-iOS/Resource/Components/BottomSheet/ Flexible/FlexibleListBottomSheet.swift

+    @State private var currentStyle: BottomSheetStyle = .half
+
+    var body: some View {
+        GeometryReader { geo in


여기도 와일드 카드!

juri123123

라이브러리로 빼니 깔끔해서 좋네요 !! 고생많으셨습니다 !

juri123123 · 2025-02-10T13:53:24Z

Spoony-iOS/Spoony-iOS/Resource/Components/BottomSheet/ Flexible/FlexibleListBottomSheet.swift

+    @State private var currentStyle: BottomSheetStyle = .half
+
+    var body: some View {
+        GeometryReader { geo in


GeometryReader를 쓰는 이유가 몰까요? 저번에 사용하지 않는 방향으로 수정하지 않았었나요. ?!!? 정말 기억이 나지 않아 물어봅니다...

GeometryReader는 바텀시트의 확장조건 ( .mini .half .full) 의 상황이 비율로 움직이고 있었는데 이 비율에 대한 부분을 감지하기 위해 사용되었습니다. 이부분은 라이브러리로 구현을 해놓은 상태이구요.
현재는 바텀시트가 full 인 상황에서 스크롤뷰를 움직일때 스크롤뷰의 스크롤 영역조절을 위해 작성을 해놧었는데,, 필요가 없었네요!
현재 geo 매개변수가 사용되지 않는것도 스크롤뷰처리를 따로 하지 않아도 되기 때문에 사용자체가 될곳이 없습니다.
해당 부분 통으로 삭제하도록 할게요!

juri123123 · 2025-02-10T13:54:36Z

Spoony-iOS/Spoony-iOS/Resource/Components/BottomSheet/ScrollOffsetPreferenceKey.swift

+
+import SwiftUI
+
+struct ScrollOffsetPreferenceKey: PreferenceKey {


사용하는 곳이 어디인가용

해당 코드도 필요없는게 맞네요! 라이브러리에 포함이 되어있어서 내부에서는 사용하지 않아도 됩니다

ㅇ

ChoiAnYong · 2025-02-11T05:42:11Z

Spoony-iOS/Spoony-iOS/Source/Feature/SearchLocation/SearchLocation.swift

-                print("Failed to fetch data:", error)
-                isLoading = false
-            }
+            await viewModel.fetchLocationList(locationId: locationId)


error throw 안되는 부분인가요?

hooni0918 added 12 commits February 7, 2025 23:13

add: #178 - flexSheet 라이브러리 도입

3f00343

chore: #178 불필요코드 1차삭제

925fd3a

feat: #178 드래그제스쳐 라이브러리와 싱크 완료

9695aa4

초록색 부분은 가독성용

feat: #178 섹션 half 경우 2개만 보이게

917431b

feat: #178 기본으로 원복

52cbc5c

feat: #178 mini 상태 구현

ea0d25e

feat: #178 2개만 렌더링하는 로직 삭제

d5d99e4

feat: #178 바텀시트의 경우에도 내부내용 보이게 수정

1dbaf44

feat: #178 검색의 경우에도 라이브러리 적용

f664825

feat: #178 기존 중복코드 삭제 및 정위치

120ef0e

feat: #178 위치이동

25ceddf

feat: #178 컨벤션

9dd53da

hooni0918 added feat 기능구현시 사용 Jihoon 나는지훈 labels Feb 9, 2025

hooni0918 requested review from juri123123, thingineeer and ChoiAnYong February 9, 2025 18:36

hooni0918 self-assigned this Feb 9, 2025

thingineeer approved these changes Feb 10, 2025

View reviewed changes

juri123123 approved these changes Feb 10, 2025

View reviewed changes

hooni0918 added 2 commits February 10, 2025 23:45

feat: #178 코리반영

aa86417

ㅇ

feat: #178 코리반영

128c3bc

ChoiAnYong approved these changes Feb 11, 2025

View reviewed changes

hooni0918 merged commit 899a952 into juri Feb 11, 2025

hooni0918 deleted the feat/#178-bottomsheetLibary branch February 11, 2025 07:00

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feat] #178-bottomsheet라이브러리로 구현 #181

[Feat] #178-bottomsheet라이브러리로 구현 #181

hooni0918 commented Feb 9, 2025 •

edited

Loading

thingineeer left a comment •

edited

Loading

thingineeer Feb 10, 2025

hooni0918 Feb 10, 2025

thingineeer Feb 10, 2025

thingineeer Feb 10, 2025

hooni0918 Feb 10, 2025

thingineeer Feb 10, 2025

juri123123 left a comment

juri123123 Feb 10, 2025

hooni0918 Feb 10, 2025

juri123123 Feb 10, 2025

hooni0918 Feb 10, 2025

ChoiAnYong Feb 11, 2025


		import SwiftUI

		struct ScrollOffsetPreferenceKey: PreferenceKey {

[Feat] #178-bottomsheet라이브러리로 구현 #181

[Feat] #178-bottomsheet라이브러리로 구현 #181

Conversation

hooni0918 commented Feb 9, 2025 • edited Loading

🔗 연결된 이슈

📄 작업 내용

동작 내용은 기존과 동일해서 생략합니다.

💻 주요 코드 설명

👀 기타 더 이야기해볼 점

thingineeer left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

juri123123 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hooni0918 commented Feb 9, 2025 •

edited

Loading

thingineeer left a comment •

edited

Loading