Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request refactors the notice section by introducing a specialized NoticeDataTable and an ArticleNavigation component, while also updating the UI styling and translating several labels into Korean for better consistency. Feedback includes addressing redundant navigation logic in the data table, fixing a CSS class typo, removing an unused filter component, and translating navigation labels to Korean to match the rest of the page.
| key={row.id} | ||
| data-state={row.getIsSelected() && 'selected'} | ||
| className="border-line cursor-pointer border-b hover:bg-transparent" | ||
| onClick={linked ? () => router.push(href) : undefined} |
There was a problem hiding this comment.
linked가 true일 때 TableRow에 onClick 이벤트가 설정되어 있고, 그 내부의 TableCell에서도 Link 컴포넌트를 사용하고 있습니다. 이는 중복된 내비게이션 로직이며, 클릭 이벤트가 버블링되어 의도치 않은 동작을 유발할 수 있습니다. TableRow의 onClick을 제거하거나, 특정 셀에만 Link를 적용하는 방식으로 정리가 필요해 보입니다.
| <ContestTitleFilter | ||
| title="State" | ||
| options={[]} | ||
| resetPageIndex={() => {}} | ||
| /> |
There was a problem hiding this comment.
ContestTitleFilter가 빈 옵션(options={[]})으로 렌더링되고 있습니다. 현재 페이지에서 필터 기능이 필요하지 않다면 제거하는 것이 좋으며, contest 디렉토리의 컴포넌트를 직접 참조하는 대신 공통 컴포넌트로 분리하여 사용하는 것을 권장합니다.
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
1 participant
Description
notice_main page를 리디자인합니다
searchbar 컴포넌트 리팩토링 진행하고 있는 것으로 알고 있어서 나중에 다시 수정하겠습니다!
Additional context
Before submitting the PR, please make sure you do the following
fixes #123).closes TAS-2667