Skip to content

Conversation

@handje
Copy link
Collaborator

@handje handje commented Feb 3, 2025

Pull request

Related issue

Resolve #111 @handje

Motivation and context

  • 사이드바 fixed 속성 수정
  • 사이드바 배치

Solution

  1. SidebarInset 내부에 메인 콘텐츠 구현
      <SidebarInset className="flex flex-col min-w-0 min-h-0 w-full h-full">
        <ChatHeader />
        <ChatSection />
      </SidebarInset>
  1. ChatHeader 내부에 sidebar trigger배치한 뒤 헤더 추가 스타일링은 따로 하지 않았습니다.
  2. content인 채팅 리스트와 chat-text-area부분의 padding값을 주어 textarea와 sidebar간격을 주어야하지만, 이부분은 채팅 레이아웃에 포함되어 따로 진행하지 않았습니다.
  3. @KimKyuHoi data는 채널이 api 또는 웹소켓으로 들어오기때문에 별도 파일로 구현하지 않고 임시로 두었습니다.
    해당 로직은 워크스페이스를 진행하며 구현할 예정입니다. a링크 또한 대체될 예정입니다.

How has this been tested

image
image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the docs/CONTRIBUTING.md document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@handje handje requested a review from KimKyuHoi February 3, 2025 12:58
@handje handje self-assigned this Feb 3, 2025
@handje handje added 🐛 BugFix 버그 수정 ✨ Feature 기능 추가 🔧 Build 쓸모없는 주석 제거 및 필요 없는 코드 제거 💄 Design 프론트엔드 CSS 관련 이슈 🖥️ FE 프론트엔드 무조건 스프린트내에 해야하는 것들 labels Feb 3, 2025
@handje handje added this to the 주톡피아 마일스톤2 milestone Feb 3, 2025
@handje
Copy link
Collaborator Author

handje commented Feb 3, 2025

@KimKyuHoi 채팅전체 레이아웃(헤더, 내부패딩 등)을 진행해야하는데 직접 진행하시는게 편하실까요? 아니면 제가 진행할까요?

Copy link
Collaborator

@KimKyuHoi KimKyuHoi left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다.

@KimKyuHoi
Copy link
Collaborator

@KimKyuHoi 채팅전체 레이아웃(헤더, 내부패딩 등)을 진행해야하는데 직접 진행하시는게 편하실까요? 아니면 제가 진행할까요?

엇 정확하게 어떤걸 말씀하시는걸까요? header같은 경우는 아마 교체하면 될것이고 내부 패딩이 어떤것인지 궁금합니다!

@handje
Copy link
Collaborator Author

handje commented Feb 4, 2025

@KimKyuHoi 채팅전체 레이아웃(헤더, 내부패딩 등)을 진행해야하는데 직접 진행하시는게 편하실까요? 아니면 제가 진행할까요?

엇 정확하게 어떤걸 말씀하시는걸까요? header같은 경우는 아마 교체하면 될것이고 내부 패딩이 어떤것인지 궁금합니다!

textarea가 사이드바와 너무 붙어있어서 padding값을 넣어야할 것 같지만 이부분은 스타일적인 부분이라 추후에 진행하겠습니다!

@handje handje merged commit 8a0fcbe into dev Feb 4, 2025
4 of 5 checks passed
@handje handje deleted the fe-fix/sidebar-style branch February 7, 2025 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

무조건 스프린트내에 해야하는 것들 🐛 BugFix 버그 수정 🔧 Build 쓸모없는 주석 제거 및 필요 없는 코드 제거 💄 Design 프론트엔드 CSS 관련 이슈 🖥️ FE 프론트엔드 ✨ Feature 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

사이드바 스타일링 에러 수정

3 participants