-
Notifications
You must be signed in to change notification settings - Fork 8
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
[FE] 피드백 모아보기 페이지 웹 접근성 추가 (#690) #692
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.
굳굳 섬세한 처리는 나중에 시간되면 해요!
<Button | ||
onClick={onClose} | ||
size="xSmall" | ||
style={{ background: "transparent", color: "black" }} |
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.
여기를 빼먹었었네요.. ㅎㅎ
<> | ||
<S.ScreenReader> | ||
{selectedFeedbackType} 리스트에 {feedbackData.length}개의 미션이 있습니다. | ||
</S.ScreenReader> | ||
|
||
<S.FeedbackCardContainer> |
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.
Fragment 를 이용하기 보다 S.ScreenReader 를 FeedbackCardContainer 하위에 놔둬도 될거 같아요!
📓 스토리북 링크
바로가기
📌 관련 이슈
✨ PR 세부 내용