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

[Init/#6] svgr 세팅 #8

Merged
merged 1 commit into from
Jul 6, 2024
Merged

[Init/#6] svgr 세팅 #8

merged 1 commit into from
Jul 6, 2024

Conversation

se0jinYoon
Copy link
Collaborator

@se0jinYoon se0jinYoon commented Jul 4, 2024

항목 내용
⏰ Time 30M
🌱 Page 공통
#️⃣ Issue Number 6

Closes #6

☀️ New-insight

  • svgr 세팅을 위해 vite-plugin-svgr 설치했습니다.
  • svgr을 컴포넌트화 하기 위해서 custom.d.ts 에 svg 컴포넌트 타입을 지정해두었습니다.
    ( .d.ts 파일기존 js 모듈을 ts에서 사용할 수 있도록 타입을 정의할 수 있는 파일 )
declare module '*.svg?react' {
  import React = require('react');
  export const ReactComponent: React.FC<React.SVGProps<SVGSVGElement>>;
  const src: string;
  export default src;
}

참고자료 - 공식 문서

⁉️ Question

📸 Screenshot

스크린샷 2024-07-04 오후 5 58 11

@se0jinYoon se0jinYoon self-assigned this Jul 4, 2024
@github-actions github-actions bot added the size/m label Jul 4, 2024
@se0jinYoon se0jinYoon changed the title init: svgr 세팅 [Init] svgr 세팅 Jul 4, 2024
@se0jinYoon se0jinYoon changed the title [Init] svgr 세팅 [Init/#6] svgr 세팅 Jul 4, 2024
@j-nary
Copy link
Member

j-nary commented Jul 4, 2024

언니 넘모 고생많았엉요 !! 확인입니당 ~
나 합세 때 svgr 설정했을 때 .d.ts 파일이 아래와 같았는데

// svg.d.ts
declare module '*.svg' {
    const content: React.FC<React.SVGProps<SVGAElement>>
    export default content
}

기존 js 모듈을 ts에서 사용할 수 있도록 타입을 정의해줄 수 있는 파일이라는 건 처음 알았네용
많이 배워갑니당 !!

Copy link
Collaborator

@ijieun ijieun left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!! svgr 을 처음 써보는데 이렇게 세팅하면 되는 군요!! 배워갑니다 :>

@yarimu
Copy link
Contributor

yarimu commented Jul 5, 2024

고생하셨습니다..!!
js+리액트 환경에서만 svgr 초기세팅을 해보았는데, ts 환경에서는 이렇게 하는 것이군요..!! 배워갑니당

Copy link
Collaborator

@lydiacho lydiacho left a comment

Choose a reason for hiding this comment

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

LGTM

@se0jinYoon se0jinYoon merged commit 54938dd into develop Jul 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

🚀 [Init] svgr 초기세팅
5 participants