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

Feature/#195 페이지 아이콘 수정 #199

Merged
merged 10 commits into from
Nov 26, 2024

Conversation

Ludovico7
Copy link
Collaborator

📝 변경 사항

🔍 변경 사항 설명

  • 모든 아이콘은 react-icons 라이브러리를 사용하도록 변경했습니다.
  • PageItem 컴포넌트에서 아이콘을 클릭하면 아이콘 수정 모달창이 렌더링됩니다.
  • 아이콘을 선택하면 PageItem의 아이콘이 변경됩니다.

🙏 질문 사항

  • 아직 변경된 내용이 서버나 클라이언트 인스턴스에 저장되지 않고 상태로만 관리되고 있습니다. 추후 현훈님 페이지 작업과 병합하여 crdt 인스턴스에 반영하도록 수정하겠습니다.

📷 스크린샷 (선택)

iconmodal.mov

✅ 작성자 체크리스트

  • Self-review: 코드가 스스로 검토됨
  • Unit tests 추가 또는 수정
  • 로컬에서 모든 기능이 정상 작동함
  • 린터 및 포맷터로 코드 정리됨
  • 의존성 업데이트 확인
  • 문서 업데이트 또는 주석 추가 (필요 시)

@Ludovico7 Ludovico7 added FE 프론트엔드 작업 Design 사용자 UI 레이아웃 디자인 및 CSS 스타일 변경 Feat 새로운 기능 추가나 기존 기능 확장 작업 labels Nov 26, 2024
@Ludovico7 Ludovico7 self-assigned this Nov 26, 2024
Copy link
Member

@pipisebastian pipisebastian left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!

Copy link
Collaborator

@minjungw00 minjungw00 left a comment

Choose a reason for hiding this comment

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

아이콘 모달 너무 멋있네요...수고많으셨습니다!

Copy link
Collaborator

@hyonun321 hyonun321 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. 회의에서 말씀해주신 내용 잘 들었습니다.

@github-actions github-actions bot merged commit 21fcf25 into dev Nov 26, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design 사용자 UI 레이아웃 디자인 및 CSS 스타일 변경 FE 프론트엔드 작업 Feat 새로운 기능 추가나 기존 기능 확장 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants