Skip to content

Conversation

@djk01281
Copy link
Collaborator

@djk01281 djk01281 commented Nov 20, 2024

🔖 연관된 이슈

📂 작업 내용

  • 사이드바 expand 기능 추가
  • 사이드바 UI 수정 (플로팅, 스크롤바 스타일링, 로고 수정)
  • 에디터 UI 수정 (플로팅)

📑 참고 자료 & 스크린샷 (선택)

scrollbar-editor-ui-ux.mp4

동작 영상

스크린샷 2024-11-21 오전 12 29 14

수정된 <Sidebar /> height

📢 리뷰 요구사항 (선택)

  • 스크롤바 좀 더 이쁘게 바꿨습니다..!
  • 에디터도 플로팅으로 바꿨는데 어떤가요? 닫혔을 때는 조금 보이는게 좋은 것 같아서 우선 남겨뒀습니다
  • heightpx로 맞췄다보니까 사이드바의 스크롤 영역도 억지로 계산해서 맞췄는데 나중에 둘다 rem으로 변경해줘야 할 것 같습니다
  • <Sidebar /> 컴포넌트가 조금 더러운데.. 스크롤바가 생겼을 때는 오른쪽 패딩을 조금 줄이는 등 수정이 필요할 것 같아서 우선 여기까지 해뒀습니다..

@djk01281 djk01281 added 🐰🐣 FE 프론트엔드 관련 이슈/PR 추적 라벨 ✨ Feature 기능 추가 및 구현, 제안과 관련된 경우 labels Nov 20, 2024
@djk01281 djk01281 requested a review from yewonJin November 20, 2024 15:25
@djk01281 djk01281 self-assigned this Nov 20, 2024
Copy link
Collaborator

@yewonJin yewonJin left a comment

Choose a reason for hiding this comment

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

👏👏

@yewonJin yewonJin merged commit be1d711 into develop Nov 21, 2024
9 checks passed
@Tolerblanc Tolerblanc deleted the feature-fe-#222 branch November 25, 2024 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐰🐣 FE 프론트엔드 관련 이슈/PR 추적 라벨 ✨ Feature 기능 추가 및 구현, 제안과 관련된 경우

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants