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

Design/#204 BoardBar 디자인 변경 #208

Merged
merged 4 commits into from
Oct 31, 2023
Merged

Design/#204 BoardBar 디자인 변경 #208

merged 4 commits into from
Oct 31, 2023

Conversation

navyjeongs
Copy link
Member

@navyjeongs navyjeongs commented Oct 30, 2023

🤠 개요

  • closes: Design: BoardBar 디자인 변경 #204
  • BoardBar의 디자인을 수정했어요.
  • 공지사항 컨테이너의 크기를 고정시켜서 공지사항이 많으면 스크롤이 생기도록 했어요.

💫 설명

  • BoardBar의 디자인을 수정했어요.

  • 공지사항 컨테이너의 크기를 고정시켜서 공지사항이 많으면 스크롤이 생기도록 했어요.

  • 일단 BoardBar관련 컴포넌트를 어느정도 수정했는데 BoardBody.tsx의 내용이 너무 많아서 분리 후에 나머지 디자인을 전부 변경해야할 것 같아요.

  • 또, BoardBody 컴포넌트 뿐만 아니라 다른 컴포넌트들도 리팩토링이 많이 필요해보여요!! 기능을 개발하면서 하나씩 리팩토링을 같이 해야할 것 같아요.

📷 스크린샷 (Optional)

Screenshot 2023-10-31 at 12 15 44 AM

@navyjeongs navyjeongs added the ✒️ Design CSS 등 사용자 UI 디자인 변경 label Oct 30, 2023
@navyjeongs navyjeongs self-assigned this Oct 30, 2023
@navyjeongs navyjeongs linked an issue Oct 30, 2023 that may be closed by this pull request
1 task
Copy link
Member

@pp449 pp449 left a comment

Choose a reason for hiding this comment

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

확실히 기능이 하나씩 계속 추가되다보니까 한 파일에 많은 내용이 들어가서 코드 가독성도 떨어지고 사이드 이펙트가 발생하는거 같아요 🥲
기능 추가가 완료되는데로 많은 부분이 리팩토링이 필요한거 같아요

@navyjeongs navyjeongs merged commit 6a0d532 into dev Oct 31, 2023
1 check passed
@navyjeongs navyjeongs deleted the Design/#204 branch October 31, 2023 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✒️ Design CSS 등 사용자 UI 디자인 변경
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Design: BoardBar 디자인 변경
2 participants