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

[#28 feature] 텍스트 필드 추가 #39

Merged
merged 10 commits into from
Jun 29, 2024
Merged

Conversation

Ahn-seokjoo
Copy link
Collaborator

@Ahn-seokjoo Ahn-seokjoo commented Jun 29, 2024

개요

이슈 링크 혹은 PR 내용 요약

텍스트 필드 추가

작업 내용

실제 작업 내용

텍필 추가
에러 추가
라벨 추가

시연 화면 (option)

실행 스크린샷 혹은 영상 첨부

image
기능A 구현 기능B 구현 ...
...

To Reviers

리뷰어들에게 전할 말

Close

close #

@Ahn-seokjoo Ahn-seokjoo self-assigned this Jun 29, 2024
Copy link
Collaborator

@fbghgus123 fbghgus123 left a 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(
Copy link
Collaborator

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(
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Ahn-seokjoo
DoraTextFieldLabel 이라고 해주면 안되냐 ?!

Comment on lines 47 to 49
val TextFieldBackGroundColor = DoraColorTokens.G1
val TextFieldHintTextColor = DoraColorTokens.G4
val TextFieldClearButtonColor = DoraColorTokens.G4
Copy link
Collaborator

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 {
Copy link
Collaborator

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>
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Ahn-seokjoo
사용되는 곳을 prefix 붙여주셈

Copy link
Member

@ddyeon ddyeon left a comment

Choose a reason for hiding this comment

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

LGTM 킹왕짱 stonejoo

@Ahn-seokjoo Ahn-seokjoo merged commit 11c74f1 into develop Jun 29, 2024
2 checks passed
@Ahn-seokjoo Ahn-seokjoo deleted the feature/save_link branch June 29, 2024 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants