feat(content): 에세이 공개 전환과 이력서 그리드 갱신 - #114
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3ccc3d9d1f
ℹ️ 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".
|
|
||
| return ( | ||
| <main className="bg-[var(--color-bg-primary)] py-12 md:py-16"> | ||
| <main className="bg-canvas py-8 text-ink md:py-10"> |
There was a problem hiding this comment.
In the resume route, this switch to bg-canvas/text-ink relies on Tailwind color tokens that are not defined in the repo: I checked styles/tokens.css and tailwind.config.js, which define --color-bg-primary, --color-text-primary, --color-border, etc., but no --color-canvas or --color-ink. Because Tailwind will not emit CSS for unknown color utilities, the new resume layout loses its intended background/text/border colors across /resume instead of using the existing design tokens.
Useful? React with 👍 / 👎.
|
|
||
| 같은 도구를 써도 누군가는 훨씬 빠르게 자기 것으로 만들고, 누군가는 그저 소비하고 끝나요. | ||
| 저는 그 차이가 답변의 속도보다 질문의 순서에서 나온다고 생각해요. | ||
| [`기술 블로그를 일주일 만에 만들 수 있었던 것`](/blog/blog-system-building)도 |
There was a problem hiding this comment.
This link is now exposed from a public post, but blog-system-building is still visibility: "private" in posts/블로그-시스템-구축기/meta.json, and getFeedData returns null for private posts unless includePrivate is set. In production, readers who click this reference from the newly published article land on an unavailable /blog/blog-system-building page, so the target should be published too, removed, or replaced with a public URL.
Useful? React with 👍 / 👎.
| {workingPrinciples.map((principle) => ( | ||
| <article | ||
| key={principle.title} | ||
| className="grid gap-2 border-b border-canvas-border pb-5 last:border-b-0 last:pb-0 sm:grid-cols-[11rem_minmax(0,1fr)] sm:gap-6" |
There was a problem hiding this comment.
AGENTS.md for this repo explicitly says not to use arbitrary Tailwind values such as p-[13px]; this new sm:grid-cols-[11rem_minmax(0,1fr)] pattern introduces a custom one-off layout value, and similar 8rem variants are copied later in the same component. Please move this to an existing pattern/token or a named CSS/module rule so the resume grid remains maintainable under the documented styling rules.
Useful? React with 👍 / 👎.
| </section> | ||
|
|
||
| <div className="grid gap-y-14 py-12 lg:grid-cols-6 lg:gap-x-10 lg:py-16"> | ||
| <aside className="order-last space-y-10 lg:order-first lg:col-span-2"> |
There was a problem hiding this comment.
The new ADR says mobile and assistive-technology users should read the career content before the supplemental profile/skills content, but this DOM still renders the entire aside before the Experience/Selected Work column and only changes visual order with CSS. Screen readers follow DOM order rather than order-first/order-last, so those users encounter Profile, Skills, Writing, Education, Certifications, and Background before the recent career sections.
Useful? React with 👍 / 👎.
* feat(content): publish revised essays and resume * fix(site): address PR review feedback --------- Co-authored-by: 박은우 <noah@bag-eun-uui-MacBookAir.local>
* feat(content): publish revised essays and resume * fix(site): address PR review feedback --------- Co-authored-by: 박은우 <noah@bag-eun-uui-MacBookAir.local>
* feat(content): publish revised essays and resume * fix(site): address PR review feedback ---------
* feat(content): publish revised essays and resume * fix(site): address PR review feedback ---------
* feat(content): 에세이 공개 전환과 이력서 그리드 갱신 (#114) * feat(content): publish revised essays and resume * fix(site): address PR review feedback --------- * docs(readme): fix formatting and remove Three.js Updated the README to correct link formatting and removed the Three.js section. * refactor(site): 탐색과 읽기 경험 단순화 (#115) * feat(content): 에세이와 이력서 공개 개선 * fix(site): PR 리뷰 피드백 반영 * refactor(blog): 비공개 WebGL 시각화 제거 * docs(content): Three.js 구현 서사 정리 * refactor(site): 전역 내비게이션과 조작 요소 단순화 * refactor(home): 아카이브 탐색 흐름 단순화 * style(site): Graphite Ink 편집 표면 적용 * docs(adr): 사이트 단순화 결정 기록 * refactor(site): 모바일 읽기 흐름 개선 모바일 헤더와 메뉴를 텍스트 중심으로 단순화하고 홈 아카이브에서 날짜와 제목의 읽기 순서를 조정했습니다.\n\nADR 0035에 모바일 아카이브와 메뉴의 반복 규칙을 기록했습니다. * refactor(blog): 콘텐츠 공개 기준 정리 공개 후보 글을 현재 구조와 검증 기준에 맞춰 리라이팅했습니다. Redis·Flink 학습 시리즈는 llm-wiki raw·draft로 이관하고 Ark에서 제거했습니다. --------- * refactor(site): 홈과 Archive 중심 구조 정리 홈을 철학 문장과 핵심 경로로 축소하고, 게시글 탐색은 별도 Archive로 분리했습니다. 2-6-4 shell, 모바일 탐색, 글 목록 행, 기존 /blog redirect를 함께 정리했습니다. * refactor(resume): 공개 이력을 요약 그리드로 재구성 회사와 팀 맥락, 프로젝트의 간략 소개·상세 설명·직접 회고 링크만 남겼습니다. 개인 CTR 파이프라인과 데이터 플랫폼 역량을 함께 보여주도록 정리했습니다. * refactor(ui): 경로 오류를 텍스트 상태로 단순화 공통 EmptyState 대신 콘텐츠 열에 맞는 상태 표식과 텍스트 재시도 행동을 사용합니다. * docs(adr): 홈과 Resume 정보 구조 결정 기록 홈·Archive shell, Resume 요약과 회고 링크, 경로 오류 상태의 선택 기준을 ADR로 남기고 인덱스를 갱신했습니다. * test(e2e): stabilize archive spacing check
* feat(content): 에세이 공개 전환과 이력서 그리드 갱신 (#114) * feat(content): publish revised essays and resume * fix(site): address PR review feedback --------- * docs(readme): fix formatting and remove Three.js Updated the README to correct link formatting and removed the Three.js section. * refactor(site): 탐색과 읽기 경험 단순화 (#115) * feat(content): 에세이와 이력서 공개 개선 * fix(site): PR 리뷰 피드백 반영 * refactor(blog): 비공개 WebGL 시각화 제거 * docs(content): Three.js 구현 서사 정리 * refactor(site): 전역 내비게이션과 조작 요소 단순화 * refactor(home): 아카이브 탐색 흐름 단순화 * style(site): Graphite Ink 편집 표면 적용 * docs(adr): 사이트 단순화 결정 기록 * refactor(site): 모바일 읽기 흐름 개선 모바일 헤더와 메뉴를 텍스트 중심으로 단순화하고 홈 아카이브에서 날짜와 제목의 읽기 순서를 조정했습니다.\n\nADR 0035에 모바일 아카이브와 메뉴의 반복 규칙을 기록했습니다. * refactor(blog): 콘텐츠 공개 기준 정리 공개 후보 글을 현재 구조와 검증 기준에 맞춰 리라이팅했습니다. Redis·Flink 학습 시리즈는 llm-wiki raw·draft로 이관하고 Ark에서 제거했습니다. --------- * refactor(site): 홈과 Archive 중심 구조 정리 홈을 철학 문장과 핵심 경로로 축소하고, 게시글 탐색은 별도 Archive로 분리했습니다. 2-6-4 shell, 모바일 탐색, 글 목록 행, 기존 /blog redirect를 함께 정리했습니다. * refactor(resume): 공개 이력을 요약 그리드로 재구성 회사와 팀 맥락, 프로젝트의 간략 소개·상세 설명·직접 회고 링크만 남겼습니다. 개인 CTR 파이프라인과 데이터 플랫폼 역량을 함께 보여주도록 정리했습니다. * refactor(ui): 경로 오류를 텍스트 상태로 단순화 공통 EmptyState 대신 콘텐츠 열에 맞는 상태 표식과 텍스트 재시도 행동을 사용합니다. * docs(adr): 홈과 Resume 정보 구조 결정 기록 홈·Archive shell, Resume 요약과 회고 링크, 경로 오류 상태의 선택 기준을 ADR로 남기고 인덱스를 갱신했습니다. * test(e2e): stabilize archive spacing check
1 participant
변경 내용
의도
master의 ADR 번호 체계를 유지해 기존 결정과 충돌하지 않게 합니다.영향 범위
/resume의 데스크톱·모바일 레이아웃이 변경됩니다.검증
npm run verify:docsnpm run test:components(53 files, 178 tests)npm run lintnpm run content:auditnpm run build/resume데스크톱·모바일, 라이트·다크 모드 확인체크리스트
codex/content-resume-refresh)에서 진행함