Skip to content

Commit 0b7d6a3

Browse files
박은우박은우
authored andcommitted
fix(site): address PR review feedback
1 parent 3ccc3d9 commit 0b7d6a3

3 files changed

Lines changed: 189 additions & 161 deletions

File tree

posts/ai-시대-학습법/index.mdx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,9 @@ AI 도구를 쓰면 모르는 개념을 설명받고, 예제 코드를 만들고
2727
깨지고, 빌드가 실패하고, 구조가 지저분해지는 과정에서 배워야 할 지점이
2828
드러나기 때문이다.
2929

30-
[`기술 블로그를 일주일 만에 만들 수 있었던 것`](/blog/blog-system-building)
31-
그런 흐름에서 시작했다. 처음부터 Next.js App Router, MDX 파이프라인, 목차
32-
생성, 읽기 진행도, 콘텐츠 메타데이터 구조를 모두 이해한 상태는 아니었다.
30+
기술 블로그를 짧은 기간에 만들던 과정도 그런 흐름에서 시작했다. 처음부터 Next.js
31+
App Router, MDX 파이프라인, 목차 생성, 읽기 진행도, 콘텐츠 메타데이터 구조를 모두
32+
이해한 상태는 아니었다.
3333
먼저 원하는 읽기 경험을 정하고, 그 경험을 실제로 만들면서 질문을 뽑아냈다.
3434

3535
- MDX 콘텐츠를 어디에서 읽어야 빌드와 런타임 책임이 분리되는가?

resume/ui/pages/ResumePage.test.tsx

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,5 +19,21 @@ describe('ResumePage', () => {
1919
screen.getByRole('heading', { name: 'Selected Work' })
2020
).toBeVisible();
2121
expect(container.querySelector('header')).toHaveClass('md:grid-cols-6');
22+
expect(container.querySelector('main')).toHaveClass(
23+
'bg-bg-primary',
24+
'text-text-primary'
25+
);
26+
});
27+
28+
it('renders career content before supplemental details in the DOM', () => {
29+
render(<ResumePage />);
30+
31+
const experience = screen.getByRole('heading', { name: 'Experience' });
32+
const profile = screen.getByRole('heading', { name: 'Profile' });
33+
34+
expect(
35+
experience.compareDocumentPosition(profile) &
36+
Node.DOCUMENT_POSITION_FOLLOWING
37+
).toBe(Node.DOCUMENT_POSITION_FOLLOWING);
2238
});
2339
});

0 commit comments

Comments
 (0)