Skip to content

[Refactor] 나의 히스토리 뷰 리팩토링#194

Merged
kwonsaebom merged 5 commits intodevelopfrom
refactor/#193/myHistory
Sep 22, 2025
Merged

[Refactor] 나의 히스토리 뷰 리팩토링#194
kwonsaebom merged 5 commits intodevelopfrom
refactor/#193/myHistory

Conversation

@kwonsaebom
Copy link
Copy Markdown
Contributor

@kwonsaebom kwonsaebom commented Sep 19, 2025

💡 Summary

close #193

✅ Tasks

  • 레이아웃 토큰 적용
  • dangerouslySetInnerHTML 제거
  • import 문 정리

👀 To Reviewer

📸 Screenshot

@vercel
Copy link
Copy Markdown

vercel bot commented Sep 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
ninedot Ready Ready Preview Comment Sep 21, 2025 6:58am

@kwonsaebom kwonsaebom added the 🛠️ Refactor 코드 리팩토링 label Sep 19, 2025
@github-actions
Copy link
Copy Markdown

빌드 결과

빌드 성공 🎉

@github-actions
Copy link
Copy Markdown

🎨 스토리북 배포 완료!

👉 스토리북 미리보기

변경된 컴포넌트의 디자인을 확인해주세요.

Copy link
Copy Markdown
Collaborator

@jisooooooooooo jisooooooooooo left a comment

Choose a reason for hiding this comment

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

다른 부분은 다 괜찮은데 스타일 수정만 조금 필요할 것 같아요!
상단 목표 박스 descriptionContainer에 상단 마진, 패딩값이 피그마랑 좀 달라서 수정이 필요해 보이고 하단에 스트릭 그리드 gridContainer에 지금은 width: '100%', maxWidth: '84.6rem' 요렇게 되어 있어서 넓이가 살짝 달라지는 문제가 있어서 피그마 확인하고 수정하면 될 것 같아요!
고생 많으셨습니둥 !!!☃︎

스크린샷 2025-09-21 오후 2 32 37

@github-actions
Copy link
Copy Markdown

🎨 스토리북 배포 완료!

👉 스토리북 미리보기

변경된 컴포넌트의 디자인을 확인해주세요.

@github-actions
Copy link
Copy Markdown

빌드 결과

빌드 성공 🎉

Copy link
Copy Markdown
Contributor

@shinjigu shinjigu left a comment

Choose a reason for hiding this comment

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

전체적으로 레이아웃 토큰 적용, dangerouslySetInnerHTML 제거 등 나의 히스토리 뷰에 관한 리팩토링 잘 진행해 주셨네요 !! 수고하셨습니다 🤍🤍

@kwonsaebom kwonsaebom merged commit f163b25 into develop Sep 22, 2025
6 checks passed
@kwonsaebom kwonsaebom deleted the refactor/#193/myHistory branch September 22, 2025 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor]: 나의 히스토리 뷰 리팩토링

3 participants