likelion-reactproject-lab12 / together

타잉 반응형 프로젝트
https://together-taing.netlify.app/
0 stars 6 forks source link

Svg 컴포넌트에 관한 제안 #49

Closed sykang013 closed 1 year ago

sykang013 commented 1 year ago

Image

Image

Image

senasoon commented 1 year ago

수정하고 커밋하겠습니다!

imironjin commented 1 year ago

@senasoon 혹시 접근성 관련도 업데이트해서 커밋 가능 할까요?

senasoon commented 1 year ago

@imironjin aria-label 값을 id값으로 주면 될까요? 예를 들어 id가 cancel-default면 aria-label="cancel-default"가 됩니다. 아니면 Svg 컴포넌트에서 props로 값을 받을까요?

imironjin commented 1 year ago

@senasoon sprite.svg의 id들이 해당 alt값으로 맞다면 그렇게 써도 괜찮을 것 같습니다.

senasoon commented 1 year ago

@imironjin 수정했습니다 :)

sykang013 commented 1 year ago

감사합니다! 👍 👍 다음 PR에 반영해서 올려주세요! 이슈는 닫겠습니다.