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

[Feature/#15] 공통 컴포넌트 textfield UI 구현 #16

Merged
merged 22 commits into from
Jan 9, 2025

Conversation

Eonji-sw
Copy link
Member

@Eonji-sw Eonji-sw commented Jan 8, 2025

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

  • merge할 브랜치의 위치를 확인해 주세요.(main❌/develop⭕)
  • 리뷰가 필요한 경우 리뷰어를 지정해 주세요.
  • 리뷰는 PR이 올라오면 최대한 빠르게 진행합니다.
  • P1 단계의 리뷰는 빠르게 확인 후 반영합니다.
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다.

📌 𝗜𝘀𝘀𝘂𝗲𝘀

📎 𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

  • 공통 텍스트 필드 UI 구현
  • 홈 FAB 아이템 터치 영역 변경

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

image image image
sign up group calendar

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

위와 같이 세 가지 케이스의 textfield 공통 컴포넌트 구현했슴다 @youjin09222 @twogarlic
사용하실 때 textFieldType에 따라 조금씩 textfield UI가 달라집니다
스크린샷 2025-01-08 오후 6 29 43
signup의 경우 하단에 "예) 홍길동" 텍스트 나타나게 하였고, calendar의 경우 글자 입력하더라도 x 아이콘 뜨지 않도록 했습니다

또한, TextFieldType 자체에 placeholder text string으로 다 박아놨으니 설정 안하셔도 됩니다 제가 해둠요
스크린샷 2025-01-08 오후 6 31 13
사용하실 때는 위와 같이 사용하시면 됩니다

스크린샷 2025-01-08 오후 6 41 24

기본 값 확인하시고 다른 건 알아서 호출할 때 인자로 넣어주세요 group 기준으로 기본 값 설정했거등요(확인하고 수정해야 하는 것 : textFieldType, value, onValueChange, placeholderColor, textStyle, shape, cursorColor, focusedBorderColor, unfocusedWithInputBorderColor, unfocusedBorderColor, maxLength)

GUI 상 테두리 색이 총 3개 입니다. 1) 포커스 되었을 때 2) 포커스 되지 않았으며 아무 입력도 없을 때 3) 포커스 되지 않았으며 텍스트는 입력되었을 때. 각각이 아래의 변수입니다
focusedBorderColor, unfocusedWithInputBorderColor, unfocusedBorderColor

이외 ProfileImagePicker, NoostakTopAppBar, NoostakCloseAppBar, NoostakBottomButton, NoostakSnackBar도 참고해서 사용하세요 @youjin09222 @twogarlic @gaeulzzang

@Eonji-sw Eonji-sw added ✅ [MOD] 코드 수정 및 내부 파일 수정 💟 [UI] UI 작업 🍀 언지 먼지 labels Jan 8, 2025
@Eonji-sw Eonji-sw self-assigned this Jan 8, 2025
@Eonji-sw Eonji-sw requested a review from a team as a code owner January 8, 2025 09:24
Copy link
Contributor

@gaeulzzang gaeulzzang left a comment

Choose a reason for hiding this comment

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

세세한 디테일까지 놓치지 않고 잘 구현해줘서 눈물이 나 엉엉
KTLINT 돌리고 바로 머지 ㄱㄱㄱ
image

Copy link

@youjin09222 youjin09222 left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다! 꼼꼼한 구현 덕분에 공통 컴포넌트로 활용하기 좋은 구조가 되었네요. 👍👍

…-textfield-ui

# Conflicts:
#	core/build.gradle.kts
#	core/src/main/java/com/sopt/core/designsystem/component/button/NoostakBottomButton.kt
#	core/src/main/java/com/sopt/core/extension/ContextExt.kt
#	core/src/main/java/com/sopt/core/util/permission/ImagePickerLaunchers.kt
#	core/src/main/java/com/sopt/core/util/permission/rememberGalleryLauncher.kt
#	presentation/src/main/java/com/sopt/presentation/groupCreate/GroupCreateRoute.kt
@Eonji-sw Eonji-sw merged commit de95476 into develop Jan 9, 2025
1 check passed
@Eonji-sw Eonji-sw deleted the feature/#15-common-textfield-ui branch January 9, 2025 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍀 언지 먼지 ✅ [MOD] 코드 수정 및 내부 파일 수정 💟 [UI] UI 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI] : 공통 텍스트 필드 UI 구현
3 participants