Skip to content

Conversation

@vvan2
Copy link
Member

@vvan2 vvan2 commented Dec 13, 2025

ISSUE

❗ WORK DESCRIPTION

  • navigation 수정 및 작업
  • Icon 추가 (mypage 뷰), 불필요한 파일 제거
  • 기존 Mypage 패키지 수정 : mypage -> mypage/main, mypage/userInfo, mypage/petinfo, mypage/courseinfo
  • mypage 컴포넌트, 뷰 수정
  • mypage navigation,model 수정

📢 TO REVIEWERS

  • 기존 presentation/mypage 에 component,navigation,model 등 통합되어있던 구조에서 하위 main,userinfo,petinfo,courseinfo 로 세분화하여 구조를 수정했습니다.
  • 1.컴포넌트 2. 뷰 3. 네비게이션 위주로 스프린트 GUI 를 적용완료하였습니다. viewmodel 및 api 관련한 로직은 추후 토큰 관련 로직 추가후 진행하겠습니답

📸 SCREENSHOT

-.Clipchamp.10.mp4

@vvan2 vvan2 requested review from JiWoo1261 and sonms December 13, 2025 05:27
@vvan2 vvan2 self-assigned this Dec 13, 2025
@vvan2 vvan2 added 🐻주완 주완 전용 라벨 mod 🎉 큰 단위 수정 - gui 반영 등 labels Dec 13, 2025
@vvan2 vvan2 linked an issue Dec 13, 2025 that may be closed by this pull request
3 tasks
Copy link
Member

@sonms sonms left a comment

Choose a reason for hiding this comment

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

너무 고생많으셨습니다!
궁금한 점이 있어 여기에 남깁니다!

  1. 가끔 onClick이랑 onValueChanged 가 빠진 부분이 있는데 필요하지 않아서 구현하지 않고 {}만 남기신 건지 아니면 추후 API 붙이실 때 반영하실 예정인지 궁금합니다!

  2. 파라미터로 전달되는 변수들 역시 state로 묶어서 관리할 수 있을 것 같은데 나중에 확인 부탁드려용

Comment on lines +19 to +63
@Composable
fun UserGenderButton(
user: String,
isSelect: Boolean,
onClick: () -> Unit,
modifier: Modifier = Modifier,
) {
Box(
contentAlignment = Alignment.Center,
modifier = modifier
.clip(RoundedCornerShape(8.dp))
.background(
color = if (isSelect) {
PawKeyTheme.colors.primary
} else {
PawKeyTheme.colors.background
}
)
.border(
width = 1.dp,
color = if (isSelect) {
Color.Transparent
} else {
PawKeyTheme.colors.defaultMiddle
},
shape = RoundedCornerShape(8.dp)
)
.noRippleClickable(onClick)
.padding(horizontal = 68.dp, vertical = 16.dp)
) {
Text(
text = user,
color = if (isSelect) {
PawKeyTheme.colors.background
} else {
PawKeyTheme.colors.defaultMiddle
},
style = if (isSelect) {
PawKeyTheme.typography.bodyActive
} else {
PawKeyTheme.typography.bodyDefault
},
)
}
}
Copy link
Member

Choose a reason for hiding this comment

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

val backgroundColor = if (isSelect) PawKeyTheme.colors.primary else PawKeyTheme.colors.background
    val borderColor = if (isSelect) Color.Transparent else PawKeyTheme.colors.defaultMiddle
    val contentColor = if (isSelect) PawKeyTheme.colors.background else PawKeyTheme.colors.defaultMiddle
    val textStyle = if (isSelect) PawKeyTheme.typography.bodyActive else PawKeyTheme.typography.bodyDefault

Box(
        contentAlignment = Alignment.Center,
        modifier = modifier
            .clip(shape)
            .background(color = backgroundColor)
            .border(
                width = 1.dp,
                color = borderColor,
                shape = shape
            )
            .noRippleClickable(onClick)
            .padding(horizontal = 68.dp, vertical = 16.dp)
    ) {
        Text(
            text = user,
            color = contentColor,
            style = textStyle
        )
    }

이거도 묶으면 좋을 것 같은데 어떻게 생각하시나ㅓ요?
지금도 사실 보기에 나쁘지는 않아요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod 🎉 큰 단위 수정 - gui 반영 등 🐻주완 주완 전용 라벨

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[refactor] 마이페이지 뷰 개선

3 participants