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

[feat] 검색 뷰 구현 #188

Merged
merged 10 commits into from
Feb 24, 2024
Merged

[feat] 검색 뷰 구현 #188

merged 10 commits into from
Feb 24, 2024

Conversation

jihyunniiii
Copy link
Collaborator

@jihyunniiii jihyunniiii commented Feb 16, 2024

Related issue 🛠

Work Description ✏️

  • 검색 뷰를 구현하였습니다.

Screenshot 📸

Screen_recording_20240225_004507.mp4

Uncompleted Tasks 😅

  • 검색 뷰 관련 디자인 논의 진행 중 (검색 이미지 정렬)

To Reviewers 📢

  • 잠이 안 와요 ㅋ.ㅋ

Copy link
Collaborator

@HAJIEUN02 HAJIEUN02 left a comment

Choose a reason for hiding this comment

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

@+id 말고 코리 달게없어염ㅋ 완벽핑~

Copy link
Collaborator

Choose a reason for hiding this comment

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

모지 이거 걍 순서 때문에 수정한거죠??

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

순서 정렬 + 새 액티비티 생성 때문에 수정하였습니당

android:layout_height="wrap_content"
android:layout_marginTop="145dp"
android:src="@drawable/img_search_pingle_graphic"
app:layout_constraintEnd_toStartOf="@+id/gl_end"
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 아래로 @+id 밭이네여 껄껄핑

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

아놔 왜 이리 예리하신지요

<string name="my_pingle_topbar">마이핑글</string>
<string name="my_pingle_soon">홈으로 이동해\n내 주변 핑글을 찾아보세요!</string>
<string name="my_pingle_done">아직 참여한 핑글이 없어요\n첫 핑글과 함께 네트워킹을 시작해보세요!</string>
<string name="my_pingle_tab_soon">\t\t\t\t\t\t\t예정된\t\t\t\t\t\t\t\t</string>
Copy link
Collaborator

Choose a reason for hiding this comment

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

아 웃기다 \t 작렬 ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

아 이거 ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ 커스텀 안 하려고 ㅋㅋㅋㅋㅋㅋㅋㅋ 민우핑이 나중에 리팩 해줄 거에요,, (아마도)
@Doreminwoo

Copy link
Member

Choose a reason for hiding this comment

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

@@ -14,6 +14,7 @@ import org.sopt.pingle.presentation.ui.common.WebViewActivity.Companion.WEB_VIEW
fun Context.hideKeyboard(view: View) {
val inputMethodManager = getSystemService(Activity.INPUT_METHOD_SERVICE) as InputMethodManager
inputMethodManager.hideSoftInputFromWindow(view.windowToken, 0)
view.clearFocus()
Copy link
Collaborator

Choose a reason for hiding this comment

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

헐 이런게 있따니 나 안그래도 오늘 ㅜ 타프젝 작업하면서 포커스가 넘 거슬렸는데ㅠㅠ 바로 적용해야딩 우헤헤

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

음하하

Copy link
Member

@Doreminwoo Doreminwoo left a comment

Choose a reason for hiding this comment

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

고생고생핑~
코리달게 없네용 굳굳

Comment on lines 39 to 59
<ImageView
android:id="@+id/iv_search_search_pingle"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginTop="145dp"
android:src="@drawable/img_search_pingle_graphic"
app:layout_constraintEnd_toStartOf="@id/gl_end"
app:layout_constraintStart_toStartOf="@id/gl_start"
app:layout_constraintTop_toBottomOf="@id/pingle_search_search" />

<TextView
android:id="@+id/tv_search_search_description"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginTop="20dp"
android:text="@string/search_description"
android:textAppearance="@style/TextAppearance.Pingle.Body.Med.16"
android:textColor="@color/g_02"
app:layout_constraintEnd_toStartOf="@id/gl_end"
app:layout_constraintStart_toStartOf="@id/gl_start"
app:layout_constraintTop_toBottomOf="@id/iv_search_search_pingle" />
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

중앙 정렬로 수정 (키보드 올라왔을 때 안 가리도록 수정)

@jihyunniiii jihyunniiii merged commit ef26e01 into develop Feb 24, 2024
1 check passed
@jihyunniiii jihyunniiii deleted the feat-search-view branch February 24, 2024 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[feat] 검색 뷰 구현
3 participants