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

refactor: 관리자 메인 페이지에서 썸네일을 png대신 svg로 보여주도록 수정 #787

Merged
merged 5 commits into from
Feb 2, 2022

Conversation

yujo11
Copy link
Collaborator

@yujo11 yujo11 commented Feb 1, 2022

구현 기능

  • 관리자 메인 페이지에서 썸네일 이미지를 png 대신 svg로 보여준다.

공유하고 싶은 내용

  • API 명세가 변경 됨에 따라 type명 변경이 있었습니다.
    • mapImageSvg -> thumbnail
  • Drawer -> MapListItem 컴포넌트에서 dangerouslySetInnerHTML을 사용 했습니다.
    • 체프와 썬의 아이디어를 반영해보려 했으나 시간 관계상 이번 PR에는 적용하기 힘들거 같습니다ㅠㅠ 추후 리팩토링 진행하도록 하겠습니다
    • 이 부분이 마음에 걸리는데 혹시 안전하게 처리할 수 있는 방법이 있다면 코멘트로 남겨주심 감사하겠습니다

Close #763

Copy link
Collaborator

@Puterism Puterism left a comment

Choose a reason for hiding this comment

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

한 번만 눈 감아드리겠습니다... 👀 ㅋㅋㅋㅋㅋㅋㅋ
유조 수고 많으셨어요! 👏👏👏

loading="lazy"
/>
</Styled.ImageInner>
<Styled.ImageInner dangerouslySetInnerHTML={{ __html: `${thumbnail}` }} />
Copy link
Collaborator

Choose a reason for hiding this comment

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

해당 부분에 대한 리팩토링 이슈를 별도로 올려두겠습니다! 꼭 걷어내는 것으로...

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator

@SunYoungKwon SunYoungKwon left a comment

Choose a reason for hiding this comment

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

mapImageSvg자체가 사라져야겠네요.
추후에 백엔드와 협의해서 api수정 후 걷어냅시다^^

@SunYoungKwon SunYoungKwon merged commit 7e42047 into dev Feb 2, 2022
@xrabcde xrabcde deleted the feat/763-png-to-svg branch February 2, 2022 13:58
This was referenced Apr 27, 2022
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.

[FE] 관리자 메인 페이지에서 썸네일 이미지를 png 대신 svg로 보여준다.
3 participants