refactor: 내 투자종목이름에 링크 걸기 및 디자인 수정 #99
Conversation
Walkthrough이번 PR은 내 계좌 페이지의 디자인과 내비게이션 로직을 수정합니다. 주식 이름에 next/link의 Link 컴포넌트를 적용하여 클릭 시 지정된 페이지로 이동할 수 있도록 변경했으며, 텍스트 색상 관련 CSS 클래스가 수정되어 스타일이 업데이트되었습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant 유저 as 사용자
participant ST as stock-table 컴포넌트
participant LN as Link (next/link)
participant PG as 대상 페이지
사용자->>ST: 주식 이름 클릭
ST->>LN: 클릭 이벤트 처리
LN->>PG: 페이지 이동 요청
Assessment against linked issues
Suggested labels
Suggested reviewers
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/app/my-account/_components/stock-table.tsx (1)
73-80: 종목명 링크 구현 확인 및 개선 제안종목명에 대한 링크 구현이 적절하게 되었습니다. URL 인코딩도 올바르게 처리되었습니다.
다만, 사용자 경험을 더욱 향상시키기 위해 다음과 같은 개선을 제안드립니다:
<Link href={`/search/${encodeURIComponent(stock.stockName)}`} - className="cursor-pointer text-16-500 hover:text-blue-600 hover:underline" + className="cursor-pointer text-16-500 hover:text-blue-600 hover:underline transition-colors duration-200" >transition 효과를 추가하여 hover 시 색상 변경이 더 자연스럽게 이루어지도록 개선할 수 있습니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/app/my-account/_components/stock-table.tsx(4 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: storybook-preview
🔇 Additional comments (3)
src/app/my-account/_components/stock-table.tsx (3)
4-4: Link 컴포넌트 import 추가 확인next/link의 Link 컴포넌트 import가 적절하게 추가되었습니다.
34-34: 테이블 헤더 스타일 변경 확인Figma 디자인 사양에 맞춰 헤더 텍스트의 font-weight가 일관되게 600에서 500으로 변경되었습니다.
Also applies to: 40-40, 46-46, 52-52, 58-58, 61-61, 64-64, 67-67
83-83: 테이블 본문 스타일 변경 확인테이블 본문의 텍스트 스타일이 디자인 사양에 맞게 일관되게 변경되었습니다.
Also applies to: 101-101, 111-111, 114-114, 118-118, 122-122
#️⃣ 이슈
📝 작업 내용
이번 PR에서 작업한 내용을 간략히 설명해주세요.
글꼴 피그마에 맞게 수정및 내 계좌에 투자한종목 클릭시 바로 차트페이지로 이동
📸 스크린샷
✅ 체크 리스트
👩💻 공유 포인트 및 논의 사항
Summary by CodeRabbit