Skip to content

feat(blog): Markdown 데이터 차트를 shadcn으로 표시 - #226

Merged
devy1540 merged 1 commit into
mainfrom
feat/markdown-charts
Sep 22, 2026
Merged

devy1540 merged 1 commit into
mainfrom
feat/markdown-charts

Conversation

@devy1540

Copy link
Copy Markdown
Owner

변경 내용

오딘 글의 두 정적 그래프를 Markdown chart 데이터 블록으로 교체하고, shadcn 보기 전환·툴팁과 Recharts로 표시합니다. 모바일에서는 이미지 확대 없이 축과 수치를 읽고 표로 원본 값을 확인할 수 있습니다.

  • 서버 HTML에 제목·단위·수치 표를 포함하고 차트가 화면에 가까워질 때만 그래프 모듈을 불러옵니다. 로딩 실패 시에도 표를 유지하고 재시도를 제공합니다.
  • 누적 막대와 가로 막대, 라이트/다크 색상, 진행 중인 주의 사선·별표, 표본·기간 툴팁을 지원합니다. 기존 지표와 집계 조건은 유지했습니다.
  • JSON 스키마를 검증해 누락 수치를 0으로 바꾸지 않고, 임의 CSS/코드 실행을 허용하지 않습니다. 기존 benchmark는 필요한 블록에서만 로드하며 동작은 유지합니다.
  • 작성법과 렌더링 기준을 docs/markdown-charts.md에 문서화했습니다.

검증

  • 빌드·타입 검사·lint 통과(기존 경고 16개, 오류 0개).
  • 차트 6개, 로딩/SSR 8개, SEO 5개, 날짜 6개, 사이트맵 6개 테스트와 내부 링크 검사 통과.
  • Chrome 데스크톱·390px·320px, 라이트/다크, 표/차트 전환, 툴팁, 기존 benchmark/Mermaid 확인.
  • 차트 모듈을 의도적으로 차단했을 때도 표를 보존하고, 재시도로 정상 복구되는 것을 확인했습니다.
  • 초기 import graph에 그래프 렌더러가 포함되지 않고, SSR에 차트 JSON 대신 수치 표가 출력되는 것을 검사했습니다.

@github-actions github-actions Bot added documentation Improvements or additions to documentation target: main PR targets the main branch area: content Blog post content changes area: blog Blog listing, post rendering, search, tags, or series area: ui Shared UI, layout, or styling area: i18n Localization and translations area: build Build, tooling, or project configuration area: ci GitHub Actions and CI configuration dependencies Dependency manifest or lockfile changes labels Sep 22, 2026
@devy1540
devy1540 merged commit 76b7cc0 into main Sep 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: blog Blog listing, post rendering, search, tags, or series area: build Build, tooling, or project configuration area: ci GitHub Actions and CI configuration area: content Blog post content changes area: i18n Localization and translations area: ui Shared UI, layout, or styling dependencies Dependency manifest or lockfile changes documentation Improvements or additions to documentation target: main PR targets the main branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant