Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#34] - 사이드바 초기 마크업 생성 및 기존 파일 컨벤션 수정 #38

Merged
merged 8 commits into from
Feb 18, 2025

Conversation

minh0518
Copy link
Member

📌 연관된 이슈 번호

close #34

🌱 주요 변경 사항

  • 사이드바 초기 마크업 구현
  • 기존 폴더 및 파일 컨벤션 적용(케밥 케이스변경, 폴더구조 수정)

📸 스크린샷 (선택)

image

🗣 리뷰어에게 할 말 (선택)

컨벤션 수정 된 부분이 있어서 우선 현재 작업된 부분 먼저 올리겠습니다
사이드바 코드 로직 및 개선 사항은 추후 곧바로 진행하겠습니다

@minh0518 minh0518 added the ✨ Feature 기능 구현 label Feb 17, 2025
@minh0518 minh0518 self-assigned this Feb 17, 2025
Copy link

vercel bot commented Feb 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
onepiece-fe ⬜️ Ignored (Inspect) Feb 17, 2025 1:29pm

@Limgabi
Copy link
Collaborator

Limgabi commented Feb 17, 2025

🚀 Storybook Deploy : https://67a21b40b49ad2ad0c08c418-twfvezzuzq.chromatic.com/


export default function TotalEvalutionPage() {
const { evaluationSummary, overallEvaluationGrade, evaluationItems } = evaluationData;
const [isSidebarOpen, setIsSidebarOpen] = useState(true);
Copy link
Collaborator

Choose a reason for hiding this comment

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

사이드바 열림 상태를 여러 컴포넌트에서 함께 사용하는 것 같은데, context로 관리하는 것도 좋은 방법일 것 같아요!

@Limgabi Limgabi merged commit 46e8e6f into develop Feb 18, 2025
5 checks passed
@Limgabi Limgabi deleted the feat-ui-sidebar branch February 18, 2025 03:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] - 사이드바 생성
2 participants