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

[Feat] 프로필 생성 UI 제작 #58

Merged
merged 28 commits into from
Feb 13, 2024
Merged

[Feat] 프로필 생성 UI 제작 #58

merged 28 commits into from
Feb 13, 2024

Conversation

ham2174
Copy link
Collaborator

@ham2174 ham2174 commented Feb 11, 2024 •
edited
Loading

관련 이슈

작업한 내용

  • 디자이너, 개발자 아이콘 변경
  • 디자인 시스템 일부 변경
    • FunchButton : Full, Large 타입 ContentVerticalPadding 피그마와 일치하게끔 변경
    • FunchTextField 4종
      • errorCaption을 텍스트 필드 내부에 포함
      • 캡션이 들어올수 있는 높이(Fixed Height) 제공
      • 에러 발생시borderLineColor가 변경되는 텍스트필드 3종(normal, maxLength, icon) Modifier내 else if 문 수정
    • FunchFeedbackButton : internal → public으로 변경
    • FunchLabel : 신규 컴포넌트 제작
  • 도메인 일부 수정
  • 프로필 생성 UI 제작 (임시)

PR 포인트

Screen_recording_20240212_220028.mp4.mp4

@ham2174
[refacor] : 디자이너, 개발자 아이콘 svg 파일로 변경
21eaac8
@ham2174
[refactor] : FunchButton [Full, Large] 타입 ContentVerticalPadding 피그마에…
45b4944 
… 맞게끔 수정
@ham2174
[refactor] : FunchTextField 4종 에러상태일때 border 바뀌게끔 수정 및 FunchErrorCapt…
886f298 
…ion 텍스트 필드내 삽입
@ham2174
[refactor] : FunchFeedbackButton Public으로 변경
067d52a
@ham2174
[feat] : 도메인 Entity Blood에 속성값으로 Blood 찾아서 반환해주는 함수 formType() 추가
c65ca5f
@ham2174
[refactor] : 도메인 Entity Profile의 subways 초기화 변경
9589a30
@ham2174
[feat] : FunchLabel 컴포넌트 제작
2ee6a25
@ham2174
[feat] : 디자인 시스템에 도메인 Entity name 따른 아이콘 찾는 함수 제작
ff31e13
@ham2174
[feat] : Profile 모듈 MbtiType UI모델 생성
0cc7409
@ham2174
[feat] : app 모듈 매니패스트 액티비티 단락에 windowSoftInputMode 추가
186329b
@ham2174
[TEMP] : (임시) 프로필 생성 UI 제작
4164aa6
@ham2174
Merge branch 'develop' of https://github.com/Nexters/Funch-AOS into f…
c663d04 
…eature/create-profile-create-ui
@ham2174 ham2174 requested a review from murjune February 11, 2024 18:37
@ham2174 ham2174 self-assigned this Feb 11, 2024
murjune
murjune approved these changes Feb 12, 2024
View reviewed changes
Copy link
Member

@murjune murjune left a comment

Choose a reason for hiding this comment

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

빡작업하느라 고생했어용~!! 👍👍👍
코멘트 몇 개만 봐주세용

core/designsystem/src/main/java/com/moya/funch/common/SearchPainter.kt Outdated Show resolved Hide resolved
core/designsystem/src/main/java/com/moya/funch/component/Button.kt Show resolved Hide resolved
feature/profile/src/main/java/com/moya/funch/CreatePofileViewModel.kt Outdated Show resolved Hide resolved
feature/profile/src/main/java/com/moya/funch/CreatePofileViewModel.kt Outdated Show resolved Hide resolved
core/domain/src/main/java/com/moya/funch/entity/Blood.kt Outdated Show resolved Hide resolved
@murjune murjune added the ham😸 Good for newcomers label Feb 12, 2024
@ham2174
Copy link
Collaborator Author

ham2174 commented Feb 12, 2024

@murjune #50 (comment) 해당 이슈들 다 해결하고 여기 PR 종료하려고 하는데 어떻게 생각해?

@murjune
Copy link
Member

murjune commented Feb 12, 2024 •
edited
Loading

: 지하철 검색시 하단 리스트 넣기
: FunchTextField에서 '홓' 이라고 입력시 innerTextFiled의 높이가 올라가 innerTextField가 움직이는 현상

: 혈액형 드롭다운 메뉴 화면 스크롤시 움직이는 현상

: 혈액형 드롭다운 메뉴가 '매칭할래요' 버튼보다 z축이 상단에 있는 현상

@ham2174 이 부분은 detail 적인 부분이여서 나중에 fix 브랜치 하나 파서 진행하는거 어떤가유??

@ham2174
Copy link
Collaborator Author

ham2174 commented Feb 12, 2024

그러면 해당 이슈들을 모은 새로운 이슈 생성하고 이 PR은 머지할게?! @murjune

@ham2174 ham2174 mentioned this pull request Feb 12, 2024
Open
6 tasks
@ham2174 ham2174 merged commit dc9eae0 into develop Feb 13, 2024
1 check passed
@ham2174 ham2174 deleted the feature/create-profile-create-ui branch February 13, 2024 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@murjune murjune murjune approved these changes

Assignees

@ham2174 ham2174

Labels
ham😸 Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feat] 프로필 생성 UI 제작
2 participants
@ham2174 @murjune