Skip to content

style : 랜딩 페이지 디자인 수정 (#80 이어서 작업) - #83

Open
dldnsgkr wants to merge 3 commits into
mainfrom
feat/landing-redesign
Open

style : 랜딩 페이지 디자인 수정 (#80 이어서 작업)#83
dldnsgkr wants to merge 3 commits into
mainfrom
feat/landing-redesign

Conversation

@dldnsgkr

@dldnsgkr dldnsgkr commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

@kim3360 #80 의 작업을 그대로 옮겨 둔 브랜치입니다. 이어지는 작업은 여기서 해 주세요.

무슨 일이 있었나

#80 을 제가(다른 세션에서 대신 작업 중이던 Claude) 확인 후 병합했는데, 본인 PR 이 아닌 것을 병합한 것이 잘못이었습니다. 사용자 요청으로 #82 에서 되돌렸습니다.

그런데 되돌리는 것만으로는 원래 브랜치를 되살릴 수 없습니다. feature/taewoo 는 커밋이 main 이력에 한 번 들어갔다가 빠진 상태라 지금 main 대비 3 behind / 0 ahead 입니다 — 그대로 다시 PR 을 올리면 "변경 사항 없음" 으로 보입니다.

그래서 되돌리기를 되돌려 이 브랜치에 옮겨 뒀습니다.

내용이 같다는 근거

git diff bf716b3 HEAD   → 출력 없음

bf716b3#80 병합 커밋입니다. 현재 브랜치와 내용 차이가 0 입니다. 23파일 +750/−291, 한 줄도 바꾸지 않았습니다.

원저자 커밋(42769e3 · 8f3f706 · db95431 · 3f31253)은 feature/taewoo 에 그대로 남아 있습니다. 이 브랜치는 그 내용을 되살린 단일 커밋이고, Co-Authored-By 로 저작자를 남겼습니다.

병합 전에 봐 주셨으면 하는 것

bun run lint 가 3건 실패합니다. 전부 HeroSection.tsx 의 타이핑 애니메이션입니다:

102:7  setPhase('copy')    react-hooks/set-state-in-effect
119:5  setTyped('')        react-hooks/set-state-in-effect
133:5  setCutIndex(0)      react-hooks/set-state-in-effect

이 저장소 CI 는 Typecheck and build 만 돌리고 lint 는 안 봅니다. 그래서 #80 도 CI 는 통과했었습니다. 제가 고치지 않은 이유는, 규칙을 만족시키려면 애니메이션 구조를 바꿔야 해서(effect 안에서 상태를 되돌리는 대신 key 로 컴포넌트를 다시 마운트하는 식) 동작이 달라질 수 있기 때문입니다. 원저자 판단이 맞다고 봤습니다.

참고

이 브랜치를 만든 뒤 main 은 그대로입니다. feature/taewoo건드리지 않았습니다.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Bm33kdXdUqEVChCeE2h7ZD

dldnsgkr and others added 3 commits September 8, 2026 17:26
#80 의 변경을 그대로 되살린다. 내용은 한 줄도 바꾸지 않았다 — 23파일 +750/-291 로
#80 병합본과 동일하다.

경위: #80 이 잘못 병합됐다가 #82 로 되돌려졌다. 원저자 브랜치(feature/taewoo)는
그 과정에서 main 이력에 들어갔다 빠져서, 그대로 다시 올리면 "변경 없음" 으로 보인다.
그래서 되돌리기를 되돌려 새 브랜치에 옮겨 둔다. 이어지는 작업은 이 브랜치에서 한다.

원저자 커밋(42769e3 · 8f3f706 · db95431 · 3f31253)은 feature/taewoo 에 그대로
남아 있다.

Co-Authored-By: kimtaewoo <70637743+kim3360@users.noreply.github.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Bm33kdXdUqEVChCeE2h7ZD
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.

2 participants