Skip to content

Conversation

@pr-citrate
Copy link
Contributor

PR 개요

세션 타임테이블에서 Ctrl+클릭으로 새 탭에서 열 수 있도록 개선했습니다.

작업 내용

기존 onClick 핸들러를 Link 컴포넌트로 변경하여 브라우저 기본 동작을 지원합니다.

변경사항:

  • onClick + navigateLink 컴포넌트 사용
  • 조건부 렌더링으로 클릭 가능한 세션만 Link로 감쌈
  • 불필요한 useNavigate import 제거

@MU-Software
Copy link
Member

안녕하세요, @pr-citrate 님! 파이콘의 프론트엔드 코드에 기여해주셔서 감사합니다! 🎉
pr-citrate님이 본 저장소의 첫 외부 기여자이십니다ㅎㅎ 😄

코드는 문제 없어 보입니다!
또한 기여해주신 내용을 고려할 시 접근성 및 사용성 측면에서 분명 navigate 대신 Link가 나을 것 같네요,
요건 @pr-citrate 님 PR을 머지한 후, 저희가 추가적으로 전부 교체하겠습니다!

다시 한번 기여 및 제안해주셔서 감사합니다! 👍

@MU-Software MU-Software merged commit d3c8e91 into pythonkr:main Aug 24, 2025
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