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

[Feature] TextField 컴포넌트 구현 #40

Merged
merged 29 commits into from
Jun 10, 2024

Conversation

ghdtjgus76
Copy link
Collaborator

🎉 변경 사항

TextField 컴포넌트 구현했습니다.

sub, success, error 토큰 색상이 변경되었고, 반응형 관련 breakpoint도 같이 추가해놨습니다.

🚩 관련 이슈

🙏 여기는 꼭 봐주세요!

Switch 컴포넌트 리팩토링이랑 SwitchGroup 컴포넌트 수정은 Checkbox 컴포넌트 머지된 후에 얼른 하겠습니다!

Copy link
Contributor

github-actions bot commented Jun 7, 2024

Copy link
Contributor

github-actions bot commented Jun 7, 2024

@eugene028
Copy link
Collaborator

흠 요것도 color contrast 때문에 a11y test 통과 못하는 듯 하네요 ㅠ

Copy link
Member

@hamo-o hamo-o left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 빠른 작업...👍

@ghdtjgus76
Copy link
Collaborator Author

흠 요것도 color contrast 때문에 a11y test 통과 못하는 듯 하네요 ㅠ

예영님께 여쭤봤는데, disabled / placeholder 관련해서는 일부러 색을 연하게 작업하신거라고 해서 별도로 색을 변경하지 않기로 했어요!
다만, 색상 대조 건에 대해서 계속 테스트가 실패하니까 텍스트 필드에 한해서 색상 대조 관련 웹 접근성 테스트는 허용하지 않도록 변경하려고 합니다!

Copy link
Contributor

github-actions bot commented Jun 7, 2024

Copy link
Contributor

github-actions bot commented Jun 7, 2024

Copy link
Collaborator

@eugene028 eugene028 left a comment

Choose a reason for hiding this comment

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

깔끔 구현~ 수고하셨어용

Copy link
Contributor

github-actions bot commented Jun 9, 2024

@ghdtjgus76 ghdtjgus76 requested review from eugene028 and hamo-o June 9, 2024 03:20
Copy link
Collaborator

@SeieunYoo SeieunYoo left a comment

Choose a reason for hiding this comment

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

수고 많으셨어용~~🙌

Copy link
Contributor

github-actions bot commented Jun 9, 2024

Copy link
Contributor

Copy link
Contributor

Copy link
Collaborator

@SeieunYoo SeieunYoo left a comment

Choose a reason for hiding this comment

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

어푸룹~~

@ghdtjgus76 ghdtjgus76 merged commit fd0be3d into main Jun 10, 2024
3 checks passed
@ghdtjgus76 ghdtjgus76 deleted the feature/text-field-component branch June 10, 2024 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants