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

[UI/YAF-66] 온보딩 생년월일 UI를 구현합니다. #22

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

Conversation

MoonsuKang
Copy link
Member

Related issue 🛠

closed #19

어떤 변경사항이 있었나요?

  • 🐞 BugFix Something isn't working
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (Junit, etc.)

CheckPoint ✅

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • PR 컨벤션에 맞게 작성했습니다. (필수)
  • merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
  • BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

Work Description ✏️

  • 작업 내용
  • 생년월일 UI 구현
  • 코드리뷰에 따른 Screen 파일 분리

Uncompleted Tasks 😅

  • Task1

To Reviewers 📢

  • Timepicker 부분에서 공통적으로 item spacing을 주고 있는데,
    스크린에서 padding을 계산할 때 해당 값을 빼서 계산하셔야 됩니다~

@MoonsuKang MoonsuKang added 📱 UI 사용자 인터페이스 관련 작업 👊 문수문수문수문수문수 labels Jan 11, 2025
@MoonsuKang MoonsuKang requested a review from DongChyeon January 11, 2025 08:33
@MoonsuKang MoonsuKang self-assigned this Jan 11, 2025
Copy link
Contributor

@DongChyeon DongChyeon left a comment

Choose a reason for hiding this comment

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

[P2]
image
폰 해상도에 따라 양력/음력 글씨가 잘릴 가능성이 있기 때문에
horizontal Padding은 최소 20으로 두되 가운데 정렬하는 것도 좋을 거 같아요.

Box(
    modifier = Modifier.fillMaxWidth(),
) {
    Box(
        modifier = Modifier
            .fillMaxWidth()
            .align(Alignment.Center)
            .padding(horizontal = 20.dp)
            .height(50.dp)
            .background(OrbitTheme.colors.gray_700, shape = RoundedCornerShape(12.dp)),
    )
    Row(
        modifier = Modifier
            .fillMaxWidth()
            .padding(20.dp),
        horizontalArrangement = Arrangement.Center,
        verticalAlignment = Alignment.CenterVertically,
    ) {
        OrbitPickerItem(
            state = lunarPickerState,
            items = lunarItems,
            visibleItemsCount = 3,
            infiniteScroll = false,
            modifier = Modifier.weight(1f),
            textModifier = Modifier.padding(8.dp),
            textStyle = OrbitTheme.typography.title2SemiBold,
            itemSpacing = itemSpacing,
        )
        //...
    }
}

@MoonsuKang
Copy link
Member Author

MoonsuKang commented Jan 11, 2025

깨달아버렸습니다.
Timepicker를 할때는 고정값을 쓰는 순간부터 고통받는다는 것을..
핸드폰의 min width를 320으로 두고 테스트 해보니 PickerItem에서 LazyColumn의 중앙값도 이상하게 찾기 시작했습니다..ㅎㅎㅎㅎ..
그래서 우선 Picker부분은 ScreenWidth를 통해 비율로 패딩을 적용하고 각 아이템을의 weight를 width(screenWidth * n)으로 계산을 하는 방식으로 처리 했는데 이것도 테스트 환경에서는 잘 됐는데 한 번 테스트 해주시면 감사하겠습니다.
저기서 텍스트 컬러가 이상한데 UI CHECK MODE의 문제 같습니다.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👊 문수문수문수문수문수 📱 UI 사용자 인터페이스 관련 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI] 온보딩 생년월일 UI를 구현합니다.
2 participants