Skip to content

feat(fe): add main page tab / 코드당 메인 페이지 - "코드당에는 어떤 기능이 있나요?" 옆에 Tap 기능 구현#3649

Open
mikeyoo0405 wants to merge 1 commit into
mainfrom
t2793-create-main-page-tab
Open

feat(fe): add main page tab / 코드당 메인 페이지 - "코드당에는 어떤 기능이 있나요?" 옆에 Tap 기능 구현#3649
mikeyoo0405 wants to merge 1 commit into
mainfrom
t2793-create-main-page-tab

Conversation

@mikeyoo0405

Copy link
Copy Markdown

코드당 메인 페이지에 "코드당에는 어떤 기능이 있나요?" 옆에 NOTICE, CONTEST, PROBELM, COURSE, STUDY 선택 버튼을 구현하였습니다

스크린샷 2026-07-20 203535

closes TAS-2793

@mikeyoo0405
mikeyoo0405 requested review from egg-zz and ojongii July 20, 2026 11:36
@mikeyoo0405 mikeyoo0405 self-assigned this Jul 20, 2026
@mikeyoo0405 mikeyoo0405 added ⛳️ team-frontend preview 이 라벨이 붙어있어야 프론트엔드 Preview 환경이 생성됩니다 labels Jul 20, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a new ServiceTabs component to allow users to switch between different service categories and integrates it into the ServiceCards component. However, there is an issue where the displayed card list (features) does not update when the selected tab changes because it is hardcoded to retrieve all features. The feedback suggests updating features to use the selected tab state so that the UI correctly filters and displays only the relevant cards.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment on lines 90 to +91
// const features = FEATURE_LIST[selectedTab]
const features = SERVICE_TABS.flatMap((tab) => FEATURE_LIST[tab])

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

현재 selectedTab 상태가 변경되어도 화면에 표시되는 카드 목록(features)이 업데이트되지 않는 문제가 있습니다. features 변수가 모든 탭의 기능을 항상 한 번에 가져오도록 설정되어 있기 때문입니다.

사용자가 선택한 탭에 해당하는 기능만 화면에 렌더링되도록 FEATURE_LIST[selectedTab]을 사용하도록 수정해야 합니다.

Suggested change
// const features = FEATURE_LIST[selectedTab]
const features = SERVICE_TABS.flatMap((tab) => FEATURE_LIST[tab])
const features = FEATURE_LIST[selectedTab]

@egg-zz egg-zz Jul 21, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

현재 각 탭마다 보여지는 디자인이 정해지지 않아 resolve 하셔도 될 것 같습니다!

@skkuding-bot

skkuding-bot Bot commented Jul 20, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: e3042928045d4d0d079314c5a273c11fbf393ecc
Health Status: Healthy

Open Preview | View in Argo CD

@mikeyoo0405
mikeyoo0405 enabled auto-merge July 21, 2026 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview 이 라벨이 붙어있어야 프론트엔드 Preview 환경이 생성됩니다 ⛳️ team-frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants