[Feat] Search view/UI #241

otoolz · 2022-11-12T01:35:26Z

구현/변경 사항

검색 관련 화면 UI
- ExploreShortcutView 탐색버튼
- 서치뷰
- 서치 결과 뷰 ( 결과 있을 때, 없을 때)

스크린샷

proposeView	SearchView	ExploreShortcutView 탐색버튼

searchBar UI

서치뷰 추천검색어 파트 추가

결과 셀 수정

서치뷰 내에 proposeView 추가

서치 뷰 내부 로직

서치바 연결 추가

서치 취소 버튼 눌렀을 때 서치 초기화면 뷰

셀 네비게이션링크 추가

JMM00

확인했습니다~! 검색 기능 추가해야할 부분을 명시해주셔서 제가 편하게 작업할 수 있었습니다~! 최고최고!!

JMM00 · 2022-11-12T08:51:47Z

HappyAnding/HappyAnding/Views/SearchView.swift

+
+            ScrollView(.horizontal) {
+                HStack {
+                    // TODO: 선택된 텍스트 검색하는 기능


제가 기능 구현해서 함께 pr올렸습니다~!

긋긋긋👍

JMM00 · 2022-11-12T08:53:27Z

HappyAnding/HappyAnding/Views/SearchView.swift

+    @State var searchText: String = ""
+    @State var searchResults: [Shortcuts] = []
+
+    let keywords: [String] = ["단축어", "갓생", "포항꿀주먹"]


추천 검색어를 쉽게 변경하기 위해서 서버에서 추천 검색어를 저장하도록 추가로 구현해볼까요?

와우 너무 좋겠는걸요?

HappyAnding/HappyAnding/Views/SearchView.swift

Co-authored-by: Elie (Jeon Jimin) <41153398+JMM00@users.noreply.github.com>

JIWON1923

LGTM!
기존 커스텀했던 x (지우기) 버튼은 없어도 될 것 같아요!

JMM00 · 2022-11-13T06:47:19Z

스크린샷에 배포용 데이터베이스가 올라와있는 것 같아서 리마인드드립니당
테스트할 때는 google plist 변경해서 작업해주세요!

HanGyeongjun

수고하셨습니다! 패딩 30으로 준 부분 로미와 확인 후 머지하면 될 것 같습니다 👍🏻

HanGyeongjun · 2022-11-13T13:39:09Z

HappyAnding/HappyAnding/Views/SearchView.swift

+
+

Suggested change

HanGyeongjun · 2022-11-13T13:49:23Z

HappyAnding/HappyAnding/Views/SearchView.swift

+                //TODO: 버튼 클릭 시 단축어 제안하는 페이지 연결
+            } label: {
+                Text("단축어 제안하기")
+                    .padding(.horizontal, 30)


4배수가 아니라서 한 번 디자이너와 확인해보면 좋을 것 같습니다
@mxnxxii

Figma상, 4배수보다 마진이 우선시되어서 4배수 적용이 되지 않은것으로 확인됩니다 :)
자세한 내용은 직접 만나서 확인 후 수정하는것으로 진행하도록 해볼게요 !!

넹 28로 적용하겠습니다

HanGyeongjun · 2022-11-13T13:53:19Z

HappyAnding/HappyAnding/Views/SearchView.swift

+                        Text(keyword)
+                            .Body2()
+                            .foregroundColor(Color.Gray4)
+                            .padding(10)


Suggested change

.padding(10)

.padding(.horizontal, 12)

.padding(.vertical, 8)

HanGyeongjun · 2022-11-13T13:56:42Z

HappyAnding/HappyAnding/Views/SearchView.swift

+                            )
+                    }
+                }
+                .padding(.leading, 16)


첫번째 셀 이후의 사이 간격은 padding이 12라서 화면 좌우 영역의 패딩과는 따로 적용되야 할 것 같습니다!

Suggested change

.padding(.leading, 16)

.padding(.leading, 12)

HanGyeongjun · 2022-11-13T14:09:40Z

여기 border 있는 뷰에서도 라인 일부가 잘리는 현상이 있네요. 제가 작성한 WriteCurationSetView에서도 동일한 문제가 발생했었는데, 이슈 작성 후 머지되면 제 뷰와 함께 수정작업해보겠습니다!
아마도 뷰 내부에 border가 그려지는 게 아니라 경계선을 중심으로 그려져서 외부에 그려진 border가 잘리는게 원인 같은데 확인해보겠습니다.

mxnxxii · 2022-11-13T16:53:06Z

HappyAnding/HappyAnding/Views/SearchView.swift

+                //TODO: 버튼 클릭 시 단축어 제안하는 페이지 연결
+            } label: {
+                Text("단축어 제안하기")
+                    .padding(.horizontal, 30)


Figma상, 4배수보다 마진이 우선시되어서 4배수 적용이 되지 않은것으로 확인됩니다 :)
자세한 내용은 직접 만나서 확인 후 수정하는것으로 진행하도록 해볼게요 !!

mxnxxii

서치 결과검색 부분 background가 white로 구현되어있는 것 같은데 확인 부탁드릴게요 :)

PR 수정 요청 사항 반영

…OSTECH/MacC-Team-HappyAnding into SearchView/UI

otoolz · 2022-11-14T08:13:12Z

수정 요청사항들 확인하고 반영했습니다~
@HanGyeongjun 확인하시면 바로 머지 하겠습니다~~

JIWON1923 and others added 10 commits November 9, 2022 22:37

[Feat] #235 - create SearchView UI

4cd1af2

Merge remote-tracking branch 'refs/remotes/origin/develop'

56a3021

[feat] searchBar UI

e5aa41b

searchBar UI

[feat] 서치뷰 추천검색어 파트 추가

3803ee2

서치뷰 추천검색어 파트 추가

[Feat] 결과 셀 수정

59f835c

결과 셀 수정

[feat] 서치뷰 내에 proposeView 추가

2a85684

서치뷰 내에 proposeView 추가

[feat] 서치 뷰 내부 로직

54e3fa4

서치 뷰 내부 로직

[feat] 서치바 연결 추가

c98405c

서치바 연결 추가

[feat] 서치 취소 버튼 눌렀을 때 서치 초기화면 뷰

872ae73

서치 취소 버튼 눌렀을 때 서치 초기화면 뷰

[feat] 셀 네비게이션링크 추가

8140a9f

셀 네비게이션링크 추가

otoolz added the Type-Feature New feature label Nov 12, 2022

otoolz added this to the 3rd Sprint milestone Nov 12, 2022

otoolz requested review from manju-minji, JMM00, JIWON1923, jim4020key, HanGyeongjun and mxnxxii November 12, 2022 01:35

otoolz self-assigned this Nov 12, 2022

JMM00 approved these changes Nov 12, 2022

View reviewed changes

Update HappyAnding/HappyAnding/Views/SearchView.swift

8d3bb7d

Co-authored-by: Elie (Jeon Jimin) <41153398+JMM00@users.noreply.github.com>

JIWON1923 approved these changes Nov 12, 2022

View reviewed changes

JMM00 mentioned this pull request Nov 13, 2022

[Feat] search 함수 구현 #243

Merged

HanGyeongjun requested changes Nov 13, 2022

View reviewed changes

mxnxxii approved these changes Nov 13, 2022

View reviewed changes

mxnxxii reviewed Nov 13, 2022

View reviewed changes

mxnxxii self-requested a review November 13, 2022 16:54

[feat] PR 수정 요청 사항 반영

5df2ddd

PR 수정 요청 사항 반영

Merge branch 'SearchView/UI' of /~https://github.com/DeveloperAcademy-P…

f20f3b8

…OSTECH/MacC-Team-HappyAnding into SearchView/UI

HanGyeongjun approved these changes Nov 14, 2022

View reviewed changes

otoolz merged commit 949d088 into develop Nov 14, 2022

HanGyeongjun deleted the SearchView/UI branch December 2, 2022 01:43

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feat] Search view/UI #241

[Feat] Search view/UI #241

otoolz commented Nov 12, 2022

JMM00 left a comment

JMM00 Nov 12, 2022

otoolz Nov 12, 2022

JMM00 Nov 12, 2022

otoolz Nov 12, 2022

JIWON1923 left a comment

JMM00 commented Nov 13, 2022 •

edited

Loading

HanGyeongjun left a comment

HanGyeongjun Nov 13, 2022

HanGyeongjun Nov 13, 2022

mxnxxii Nov 13, 2022

otoolz Nov 14, 2022

HanGyeongjun Nov 13, 2022

HanGyeongjun Nov 13, 2022

HanGyeongjun commented Nov 13, 2022

mxnxxii Nov 13, 2022

mxnxxii left a comment

otoolz commented Nov 14, 2022

[Feat] Search view/UI #241

[Feat] Search view/UI #241

Conversation

otoolz commented Nov 12, 2022

관련 이슈

구현/변경 사항

스크린샷

JMM00 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

JIWON1923 left a comment

Choose a reason for hiding this comment

JMM00 commented Nov 13, 2022 • edited Loading

HanGyeongjun 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

Choose a reason for hiding this comment

HanGyeongjun commented Nov 13, 2022

Choose a reason for hiding this comment

mxnxxii left a comment

Choose a reason for hiding this comment

otoolz commented Nov 14, 2022

JMM00 commented Nov 13, 2022 •

edited

Loading