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

[chore] 코스 상세 하단 버튼 크기 조정 #250

Merged
merged 5 commits into from
Sep 13, 2024

Conversation

t1nm1ksun
Copy link
Member

@t1nm1ksun t1nm1ksun commented Sep 11, 2024

Related issue 🛠

Work Description ✏️

  • 좋아요 버튼과 내 일정에 등록하기 버튼의 세로 길이를 일치시켰습니다!

Screenshot 📸

image image

폰트 커졌을 때 기기대응!

image

Uncompleted Tasks 😅

  • N/A

To Reviewers 📢

똑같아 보이나...? 피그마에 나온 값이 좀 이상한데 한번 더 같이 체크해봐요!

@@ -38,7 +41,7 @@ fun CourseDetailBottomBar(
onClick = onLikeButtonClicked,
cornerRadius = 14.dp,
paddingHorizontal = 23.dp,
paddingVertical = 18.dp
paddingVertical = 16.5.dp
Copy link
Contributor

Choose a reason for hiding this comment

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

이거 이렇게 패딩 값을 조절하는 게 아니라 기기에서 설정해놓은 폰트 사이즈에 따라 내 일정에 추가하기 버튼 사이즈가 커지거나 작아지면 그거에 맞춰서 좋아요 버튼 크기를 조절해달라는 말인데!
혹시 이거 확인 하셨을까요????

Copy link
Contributor

@jihyunniiii jihyunniiii left a comment

Choose a reason for hiding this comment

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

야무지다잉

@@ -22,13 +32,16 @@ fun CourseDetailBottomBar(
onLikeButtonClicked: () -> Unit,
onEnrollButtonClicked: () -> Unit
) {
var buttonHeight by remember { mutableStateOf(0.dp) } // 버튼 크기를 저장하는 상태 변수
Copy link
Contributor

Choose a reason for hiding this comment

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

주석 좀 지워주겠서??????????????????/

Comment on lines 41 to 54
paddingVertical = 18.dp
paddingVertical = 16.5.dp
Copy link
Contributor

Choose a reason for hiding this comment

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

이거 다시 18.dp로 돌려주세용

@t1nm1ksun t1nm1ksun merged commit 0100c94 into develop Sep 13, 2024
1 check passed
@t1nm1ksun t1nm1ksun deleted the chore-course-detail-button-size branch September 13, 2024 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[chore] 코스 상세 하단 버튼 크기 조정
2 participants