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] 동행 모집 화면 UI 구성 및 화면 구현 #39

Merged
merged 22 commits into from
Sep 2, 2024

Conversation

easyhooon
Copy link
Contributor

@easyhooon easyhooon commented Aug 31, 2024

image

style)
동행 모집 화면 UI 를 구성하였습니다.

feat)
동행 모집 화면내 입력 로직들을 구현하였습니다.
(작성완료 버튼 활성화 validation 관련해서 디자이너 님께 질문 올린 부분이 있는데, 해당 질문 답변 받고 추가 구현 하도록 하겠습니다 -> 완료)
홈 화면 내에 버튼을 추가하여, 동행 모집 화면으로 이동할 수 있도록 임시로 구현해놓았습니다.

problem)
TextField 수정 이후, 개인화 생년월일 텍스트 필드가 동작하지 않는 문제가 발생하여, 이를 해결해보도록 하겠습니다. -> 완료

@easyhooon easyhooon added feature New feature or request design tasks releated to design labels Aug 31, 2024
@easyhooon easyhooon self-assigned this Aug 31, 2024
@easyhooon easyhooon added the bug Something isn't working label Aug 31, 2024
This was linked to issues Aug 31, 2024
maxLength 가 null 로 설정되었을 경우, 입력된 값이 무시되는 문제 해결
@easyhooon easyhooon removed the bug Something isn't working label Sep 1, 2024
@easyhooon easyhooon force-pushed the feature/mate-recruit branch from a5516fc to 093af24 Compare September 1, 2024 10:17
@wjdtkdgns777
Copy link
Collaborator

수고하셨습니다!!
코드상으로는 괜찮았는데 실제 돌려보다보니, wheel피커가 현재시간보다 이전으로 설정 안되는게
왜 이게 다이얼이 돌아가는지 모르면 조금 어색한거같아서 설정은 되지만 완료버튼을 누를때 validation되어 이전시간이면 안되게 하는식으로 하는건 어떨까 여쭤보고싶네요!!
그리고 바텀시트 색상이 푸른빛이 도는데 지난번처럼 tripmatetheme으로 감싸거나 컬럼 fillmaxsize하면 될거같아요
마지막으로 timepicker부분에 별건 아니고 왼쪽에 : 가 뜨네요
이외에는 전 너무 잘하신거같습니다!!

@easyhooon
Copy link
Contributor Author

easyhooon commented Sep 1, 2024

마지막으로 timepicker부분에 별건 아니고 왼쪽에 : 가 뜨네요

맞습니다. 좋은 지적입니다. 관련해서 이슈도 어떤 분께서 올려주셨는데 PR 도 그렇고,한 1년 넘게 라이브러리 업데이트가 없는 것 같더군요.(아쉽) 다이얼로그가 다시 처음 상태로 돌아가는 것 동작이 부자연스러운 것도 저도 느꼈던 바 입니다.

따라서 다른 라이브러리로 교체해볼까해서 알아보고 있었고,
https://github.com/darkokoa/compose-datetime-wheel-picker
https://github.com/Chaintech-Network/compose_multiplatform_date_time_picker

두 라이브러리 중에 하나로 migration 해봐보겠습니다.

@easyhooon
Copy link
Contributor Author

https://github.com/darkokoa/compose-datetime-wheel-picker <- 해당 라이브러리는 사용했었던 라이브러리의 multiplatform 버전이라 똑같은 문제가 발생하는 군요... 두번째 라이브러리로 시도해보겠습니다

darkokoa/compose-datetime-wheel-picker#19

@wjdtkdgns777
Copy link
Collaborator

끙 아쉽스.. 근데 보니까 디자인대로 아래 패딩 60이긴 한데 이거 공백이 너무 큰거같아요 ㅋㅋㅋ
한 2~ 30으로 하자고 말해볼까요? 특히 폴드로 보니까 더 어색해서

@easyhooon
Copy link
Contributor Author

끙 아쉽스.. 근데 보니까 디자인대로 아래 패딩 60이긴 한데 이거 공백이 너무 큰거같아요 ㅋㅋㅋ
한 2~ 30으로 하자고 말해볼까요? 특히 폴드로 보니까 더 어색해서

다이얼로그로 바꾸면서 적당히 24 정도로 만들어보고 있습니다

Copy link
Collaborator

@wjdtkdgns777 wjdtkdgns777 left a comment

Choose a reason for hiding this comment

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

LGTM! 임시구현이라 뒤로가기 구현 안하신줄 알았는데
뭐 어차피 로직 같을거같아서 상관없겠군용
수고하셨습니다!!

@easyhooon easyhooon merged commit 27adcb7 into develop Sep 2, 2024
1 check passed
@easyhooon easyhooon deleted the feature/mate-recruit branch September 2, 2024 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design tasks releated to design feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

동행 모집 화면 구현 동행 모집 화면 UI
3 participants