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

[PC-434] 가치관 talk ui 구현 #43

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

sksowk156
Copy link
Contributor

1. ⭐️ 변경된 내용

가치관 talk ui 구현

2. 🖼️ 스크린샷(선택)

KakaoTalk_20250129_200805968.mp4

3. 💡 알게된 부분

4. 📌 이 부분은 꼭 봐주세요!

ui를 확인해보니까 '프로필 생성' 이랑 '프로필' 이렇게 2개가 있는데, '프로필 생성'의 RegisterProfile이랑 '프로필'의 MainProfile이 같은 profile 모듈에 있는 것 같더라구요. 이거 모듈을 분리해야 될까요??

@sksowk156 sksowk156 added UI/UX 🎨 디자인 시스템, 디자인 리소스 관련 코드 🎨 리뷰 원해요🔥 피어의 리뷰를 기다리는 ing.. 🔥 ㅈㅎ찐혁 🌙 훗날 세상을 아우를 남자, sksowk156 labels Jan 29, 2025
@sksowk156 sksowk156 requested a review from tgyuuAn January 29, 2025 11:12
@sksowk156 sksowk156 self-assigned this Jan 29, 2025
Copy link
Member

@tgyuuAn tgyuuAn left a comment

Choose a reason for hiding this comment

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

ui를 확인해보니까 '프로필 생성' 이랑 '프로필' 이렇게 2개가 있는데, '프로필 생성'의 RegisterProfile이랑 '프로필'의 MainProfile이 같은 profile 모듈에 있는 것 같더라구요. 이거 모듈을 분리해야 될까요??

일단 profile 이라는 모듈 안에 같이 넣어두면 될 것 같습니다!

쓰다가 분리해야할 것 같이 커지면 분해하도록하죠!



++ 추가적으로 디스코드에 디자이너님들이 추가적인 요구사항이 있는 것 같아요!

그 부분 해결해주시고 머지하면 될 것 같습니다~

@@ -129,7 +130,7 @@ fun PieceTextInputLong(
onValueChange = { input ->
limit?.let { if (input.length <= limit) onValueChange(input) } ?: onValueChange(input)
},
singleLine = true,
singleLine = false,
readOnly = readOnly,
keyboardOptions = KeyboardOptions(
keyboardType = keyboardType,
Copy link
Member

Choose a reason for hiding this comment

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

👍

@tgyuuAn tgyuuAn added 머지 해도될듯염🌟 현재 코드를 기존 코드에 합쳐도 될 것 같다라고 판단..! 🌟 and removed 리뷰 원해요🔥 피어의 리뷰를 기다리는 ing.. 🔥 labels Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI/UX 🎨 디자인 시스템, 디자인 리소스 관련 코드 🎨 ㅈㅎ찐혁 🌙 훗날 세상을 아우를 남자, sksowk156 머지 해도될듯염🌟 현재 코드를 기존 코드에 합쳐도 될 것 같다라고 판단..! 🌟
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants