feat: 푸터에 RSS 구독 아이콘을 추가 - #141
Merged
Merged
Conversation
- 📋 작업 내역 - `src/commons/ui/icons/Rss.tsx`: Lucide `rss`(v1.48.0, ISC) 모양을 두께 3 의 면으로 그린 아이콘. `viewBox="-1.5 -1.5 27 27"` 로 둘레에 3 만큼 여백을 둔다 - `src/commons/ui/icons/Icons.tsx`: `Icons.Rss` 등록 - `src/modules/layout/ui/Contact.tsx`: 푸터 아이콘 줄 다섯 번째에 `/rss.xml` 링크 추가 (`target="_blank"`, `aria-label="RSS 피드 구독"`) - 💡 설계 이유 - #138 로 피드를 열었지만 화면에 링크가 없었다. 레퍼런스 블로그 16곳의 서버 HTML 을 보니 화면에 둔 곳은 대부분 아이콘이었고, kciter 와 heropy 는 푸터 소셜 아이콘 줄에 두었다. 모바일 헤더는 좁아 푸터 줄 끝에 붙였다 - 누르면 `/rss.xml` 을 새 탭으로 연다. XSL 로 꾸민 안내 페이지나 주소 복사도 검토했지만, 레퍼런스 대부분이 원문으로 바로 링크하고 리더에 넣을 주소만 있으면 돼서 뺐다 - 아이콘 출처 - `@ant-design/icons-svg` 4.6.0 의 848개에는 RSS 가 없었다(`GithubFilled` 는 있어 검색은 맞다). Lucide 에는 선으로 그린 `rss` 가 있다 - 채운 Simple Icons RSS 와 나란히 그려 비교했고, 사용자가 Lucide 모양을 골랐다 - 선을 면으로 옮긴 이유: 푸터 아이콘 색은 `icons-color` 의 `fill-*` 클래스로 정해진다. Lucide 의 `stroke="currentColor"` 를 그대로 쓰면 RSS 만 글자색을 따라가고 hover 색도 바뀌지 않는다. 호는 반지름 ±1.5 의 띠와 둥근 끝, 점은 반지름 2.5 의 원으로 그렸다 - 두께 2, 2.5, 3 과 여백 0, 1.5, 3 을 옆 아이콘과 32px, 28px 로 나란히 그려 보고 사용자가 두께 3, 여백 3 을 골랐다 - 다른 링크는 `<Link>` 인데 RSS 는 `<a>` 다. `/rss.xml` 은 화면 라우트가 아니라 XML 을 돌려주는 리소스 라우트라 클라이언트 내비게이션을 타지 않게 했다 - 확인한 것 - 개발 서버 푸터를 1280px, 390px 에서 라이트와 다크, hover 로 캡처했다. 색과 hover 색이 옆 아이콘과 같고, 390px 에서 RSS 와 GitHub SVG 폭이 둘 다 28px 다 - typecheck 통과, lint 에러 0 (경고 15개는 작업 전과 같다), test 393개 통과 (`NODE_OPTIONS=--no-experimental-webstorage`)
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Wisesaturn
commented
Sep 28, 2026
Wisesaturn
left a comment
Wisesaturn
left a comment
Owner
Author
There was a problem hiding this comment.
Code Review
확신도 80 이상인 지적은 없습니다. 변경이 작아(아이콘 SVG 하나, 등록 한 줄, 링크 한 개) 단계를 줄여 직접 확인했어요.
- 이슈 #140 완료 조건: 푸터 다섯 번째 아이콘, 새 탭에서
/rss.xml, 라이트와 다크와 hover 색, 28px과 32px 크기를 캡처로 확인했습니다 - 컨벤션: 아이콘은 다른 아이콘과 같은 모양의 컴포넌트(
IconProps,className전달,fill="currentColor")이고Icons에 등록했습니다. 새 패키지는 없습니다 <a>를 쓴 것은 의도입니다./rss.xml은 리소스 라우트라<Link>의 클라이언트 내비게이션을 타면 안 됩니다- 라이선스: Lucide(ISC) 출처와 버전을
Rss.tsx주석에 적었습니다
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request Type
관련 이슈
closes #140
작업 내용
푸터에서 RSS 피드로 바로 갈 수 있습니다
/rss.xml)가 열리고, 그 주소를 RSS 리더에 넣으면 새 글을 구독할 수 있습니다.검증
href="/rss.xml",target="_blank",rel="noreferrer",aria-label="RSS 피드 구독"pnpm typecheck통과,pnpm lint에러 0 (경고 15개는 작업 전과 같음)pnpm test393개 통과 (NODE_OPTIONS=--no-experimental-webstorage. Node 26 에서 기존 다크 모드 테스트가 깨지는 문제는 이번 변경과 관계없습니다)