Skip to content

feat(blog): 글 읽기 레이아웃 개선 - #124

Merged
dev-wooyeon merged 2 commits into
mainfrom
codex/inline-toc-reading-layout
Jul 27, 2026
Merged

feat(blog): 글 읽기 레이아웃 개선#124
dev-wooyeon merged 2 commits into
mainfrom
codex/inline-toc-reading-layout

Conversation

@dev-wooyeon

@dev-wooyeon dev-wooyeon commented Jul 27, 2026

Copy link
Copy Markdown
Owner

변경 내용

  • 데스크톱 우측 Portal 목차를 글 내부 인라인 목차로 변경
  • 목차 항목이 2개 이상일 때만 표시
  • 목차 번호, 계층 들여쓰기, 활성 상태 지원
  • 콘텐츠 페이지 좌측 탐색을 sticky로 처리하고 본문 여백 조정
  • 인라인 코드 색상 토큰 분리
  • 좌측 하단 GitHub / Email / RSS 글자 크기를 text-base에서 text-sm으로 축소
  • 목차 스타일은 globals.css의 semantic class로 작성

의도

글 내부에서 목차와 본문을 자연스럽게 함께 탐색할 수 있도록 읽기 흐름을 개선하고, 짧은 글에는 불필요한 목차가 표시되지 않도록 했습니다.

영향 범위

콘텐츠 페이지의 데스크톱 탐색 레이아웃, 인라인 목차, 인라인 코드 색상, 좌측 하단 외부 링크 크기에 영향을 줍니다.

검증

  • npm run test:unit -- blog/ui/components/TableOfContents.test.tsx styles/globals.test.ts styles/tokens.test.ts — 24 tests passed
  • npm run lint
  • npm run lint:css:syntax
  • npm run build

체크리스트

  • 작업 단위 브랜치(codex/<task>)에서 진행함
  • 커밋 메시지 규칙(<type>: <한국어 설명>) 준수
  • 관련 이슈/PR 링크가 필요한 변경 아님

참고: 말하기 에세이 URL의 404는 기존 visibility: private 설정과 관련된 별도 이슈이며, 최신 main의 public 전환 및 URL percent-decoding 수정이 이번 PR의 기준에 반영되어 있습니다.

@vercel

vercel Bot commented Jul 27, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
ark Ready Ready Preview, Comment Jul 27, 2026 3:22pm

@dev-wooyeon
dev-wooyeon force-pushed the codex/inline-toc-reading-layout branch from 7f3f6ec to a79f661 Compare July 27, 2026 15:11
@dev-wooyeon dev-wooyeon changed the title feat(blog): refine article reading layout feat(blog): 글 읽기 레이아웃 개선 Jul 27, 2026
@dev-wooyeon
dev-wooyeon marked this pull request as ready for review July 27, 2026 15:11

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a79f661fe0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +46 to +47
return (
<nav className="ark-article-toc" aria-label="이 글의 목차">

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Record the inline TOC decision in an ADR

Replacing the fixed body portal with an in-flow TOC chooses between two implementations with material visibility, scrolling, and responsive-layout tradeoffs, and it also revises the reading layout previously covered by ADR 0051. This commit changes the decision without adding a numbered successor ADR or updating the ADR index; document the alternatives and consequences before merging.

AGENTS.md reference: AGENTS.md:L26-L35

Useful? React with 👍 / 👎.

Comment on lines +25 to +27
.ark-site-grid[data-page-layout='content'] .ark-article-container {
margin-right: 0;
margin-left: 0;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep the comments aligned with the article

On desktop widths where the content grid column exceeds the container's 800px maximum (including the configured 1440px desktop viewport), these overrides left-align only .ark-article-container, while the comments at BlogPostPage.tsx:178 remain in a centered Container size="md". The comments therefore begin roughly half the surplus column width to the right of the article; apply the same alignment policy to both sections or retain centering for both.

AGENTS.md reference: AGENTS.md:L93-L95

Useful? React with 👍 / 👎.

@dev-wooyeon
dev-wooyeon merged commit 4e0efb8 into main Jul 27, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant