-
Notifications
You must be signed in to change notification settings - Fork 2
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
[#28 feature] 텍스트 필드 추가 #39
Conversation
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.
@Ahn-seokjoo
👍👍👍👍👍👍
import com.mashup.dorabangs.core.designsystem.theme.ErrorLabelColorTokens | ||
|
||
@Composable | ||
fun DoraErrorLabel( |
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.
@Ahn-seokjoo
DoraTextFieldErrorLabel 해주면 안되나 ?
import com.mashup.dorabangs.core.designsystem.theme.LabelColorTokens | ||
|
||
@Composable | ||
fun DoraLabel( |
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.
@Ahn-seokjoo
DoraTextFieldLabel 이라고 해주면 안되냐 ?!
val TextFieldBackGroundColor = DoraColorTokens.G1 | ||
val TextFieldHintTextColor = DoraColorTokens.G4 | ||
val TextFieldClearButtonColor = DoraColorTokens.G4 |
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.
여기 TextField 빼도 될듯??
object에 이미 네이밍됨
val TextFieldClearButtonColor = DoraColorTokens.G4 | ||
} | ||
|
||
object LabelColorTokens { |
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.
여기는 Prefix에 TextField 추가하면 좋을 듯 ??
@@ -7,4 +7,5 @@ | |||
<string name="home_empty_feed">테스트 테스트 테스트 테스트</string> | |||
<string name="snack_bar_title">클립보드에 복사한 링크 저장</string> | |||
<string name="snack_bar_cancel_description">스낵바 취소</string> | |||
<string name="url_text_clear">url 모두 지우기</string> |
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.
@Ahn-seokjoo
사용되는 곳을 prefix 붙여주셈
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 킹왕짱 stonejoo
개요
텍스트 필드 추가
작업 내용
텍필 추가
에러 추가
라벨 추가
시연 화면 (option)
To Reviers
Close
close #