Skip to content

Conversation

@Bori-github
Copy link
Collaborator

📌 요약

  • 탭 버튼 UI 수정

📝 작업 내용

  • 탭 버튼 UI 수정

🔍 스크린샷

2025-10-31.1.22.48.mov

💡 참고 사항

  • 없음

@Bori-github Bori-github self-assigned this Oct 31, 2025
@Bori-github Bori-github added the enhancement New feature or request label Oct 31, 2025
@ohah ohah requested a review from Copilot October 31, 2025 04:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the tab UI by extracting the inline tab component into a reusable TabButton component, improving code organization and maintainability.

  • Extracted tab button UI logic into a new TabButton component in the tab feature module
  • Exported the Tab interface from the playground store to enable type sharing
  • Updated the initial tab title from "✨New Playground" to "New Tab"

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
playground-groups.tsx Refactored to use the new TabButton component instead of inline JSX
tab-button.tsx New component containing the extracted tab button UI and logic
tab/ui/index.ts Barrel export for the TabButton component
tab/index.ts Barrel export for the tab feature module
store.ts Exported the Tab interface and simplified the initial tab title

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Bori-github Bori-github merged commit 640f221 into main Nov 2, 2025
3 checks passed
ohah pushed a commit that referenced this pull request Dec 12, 2025
* refactor: 탭 버튼 컴포넌트 분리

* style: 탭 버튼 스타일 수정

* style: 탭 버튼 제목 ellipsis 스타일 추가

* style: 탭 생성 시 기본 제목 수정 및 탭 제목 텍스트 스타일 수정

* chore: 불필요한 스타일 제거
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants