-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature/#52 mypage followlist #65
Conversation
large, medium button component 추가
MVI Pattern 적용
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨어요. 컨플릭트 나는 부분 확인하셔야할거 같아요
fun getButtonAndTextColor(isFollowed: Boolean): Pair<Color, Color> { | ||
return if (isFollowed) { | ||
Pair(RecordyTheme.colors.gray08, RecordyTheme.colors.white) | ||
} else { | ||
Pair(RecordyTheme.colors.white, RecordyTheme.colors.gray08) | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
텍스트와 버튼의 컬러를 하나로 Pair 를 활용해서 받는건 생각 못해봤는데 잘 살펴보고 가요
import androidx.compose.ui.unit.dp | ||
import com.record.designsystem.theme.RecordyTheme | ||
|
||
@Composable | ||
fun RecordyButton( | ||
fun RecordyButtonLarge( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이름 변경으로 인해 conflict가 난거 같아요 rebase를 통해 하나로 통일하셔야할 거 같아요
이미 develop 에 올라온 버튼의 경우 다른 분들이 사용하고 계실 수 있으니 수정을 피하는게 좋다고 생각이 드네요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
해결완료 감사합니당
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨어요
colors = ButtonDefaults.buttonColors( | ||
containerColor = colors.first, | ||
), | ||
shape = RoundedCornerShape(8.dp), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요러한 부분들 staless하게 호이스팅해도 좋을 것 같습니다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
감사합니다!!!
modifier: Modifier = Modifier, | ||
) { | ||
Button( | ||
onClick = onClick, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CustomClickable만들어 놓은 거 사용해서 리플같은 것도 고료하면 더욱 좋을거 같습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵 감사합니당
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
using derivedState
large, medium button component 추가
📌 PR 요약
🌱 작업한 내용
SideEffect
로 상태 전달합니다🌱 PR 포인트
📸 스크린샷
2024-07-11.025216.mp4
📮 관련 이슈