Conversation
| @@ -13,18 +13,24 @@ object DoraColorTokens { | |||
| val G3 = Color(0xFFDEE2E6) | |||
| val G2 = Color(0xFFE9ECEF) | |||
| val G1 = Color(0xFFF4F6F8) | |||
| val White = Color(0xFF000000) | |||
| val White = Color(0xFFFFFFFF) | |||
|
|
||
| object DialogRoundTokens { | ||
| val Radius = DoraRoundTokens.Round16 | ||
| } |
There was a problem hiding this comment.
저는 token을 shape, color정도만 생성하는 걸로 이해했는데
컴포넌트별로 만들어주는건가유?!!!!
There was a problem hiding this comment.
오끼 근데 어디까지 만들지를 정하면 좋을 것 같넹~~ 금욜에 이야기해보자
| val ContainerColor1_Off = DoraColorTokens.G2 | ||
| val OnContainerColor_1Off = DoraColorTokens.G4 | ||
| val ContentColor_1Off = DoraColorTokens.G4 |
There was a problem hiding this comment.
@ddyeon
헉 수정하겠음
1_off인데 ....
| enabled: Boolean, | ||
| onClickButton: () -> Unit, |
There was a problem hiding this comment.
enable에 defaultValue줘도 좋을 것 같다는 느낌쓰~~!!
아직 disable경우가 적으니!!!
There was a problem hiding this comment.
@ddyeon
넴 ~
근데 오버라이딩 하면서 디폴트 값을 사용하지 못하게 돼서 넣진 않았던 것 같음
There was a problem hiding this comment.
@ddyeon
오버라이드 한 메서드라 default value가 설정 안되는 이슈 ...
There was a problem hiding this comment.
옹~~ 근데 DoraButton에는 넣을 수 있지 않을까?!!
There was a problem hiding this comment.
�override여도 default는 넣을 수 있는뎅 @composable이랑 합치면 안되는구낭 나도 몰랐넹
근데 도라버튼에 enabled를,, 흠 default value를 넣어야하낭 고민이긴 하네
넣으면 편할 것 같은데, 안넣는게 맞는 것 같기도 하고, 넣으면 편할 것 같은데... 안넣는게 맞는 것 같기도 하고...넣으면 편할 것 같은데... 안넣는게 맞는 것 같기도 하고...넣으면 편할 것 같은데... 안넣는게 맞는 것 같기도 하고...넣으면 편할 것 같은데... 안넣는게 맞는 것 같기도 하고...넣으면 편할 것 같은데... 안넣는게 맞는 것 같기도 하고...넣으면 편할 것 같은데... 안넣는게 맞는 것 같기도 하고...넣으면 편할 것 같은데... 안넣는게 맞는 것 같기도 하고...넣으면 편할 것 같은데... 안넣는게 맞는 것 같기도 하고...넣으면 편할 것 같은데... 안넣는게 맞는 것 같기도 하고...
| Text( | ||
| modifier = Modifier.padding(top = 8.dp), | ||
| text = content, | ||
| style = DoraTypoTokens.caption3Medium, | ||
| color = DialogColorTokens.ContentColor, |
There was a problem hiding this comment.
content 두 줄일 때 가운데 정렬 필수인 것 같당~~
There was a problem hiding this comment.
@ddyeon
위와 같이 수정
| enabled: Boolean, | ||
| radius: RoundedCornerShape, |
There was a problem hiding this comment.
@fbghgus123 오빠 여기에 넣을 수 있을것 같은뎅?!!
|
|
|
|
||
| @Composable | ||
| internal fun DoraButton( | ||
| modifier: Modifier, |
There was a problem hiding this comment.
이건 보통 = Modifier로 넣어주긴해
There was a problem hiding this comment.
@Ahn-seokjoo
앗 그렇구만 !
|
|
||
| @Composable | ||
| fun DoraDialog( | ||
| modifier: Modifier = Modifier, |
There was a problem hiding this comment.
다연이 피드백 받고 수정한 듯
개요
작업 내용
시연 화면 (option)
To Reviers
Close
close #