Skip to content

Comments

[fix] 헤더 미사용 기능 숨김#492

Merged
dolmeengii merged 1 commit intoKernel360:developfrom
geumyoung00:fix/unuse-func
Aug 6, 2025
Merged

[fix] 헤더 미사용 기능 숨김#492
dolmeengii merged 1 commit intoKernel360:developfrom
geumyoung00:fix/unuse-func

Conversation

@geumyoung00
Copy link
Collaborator

@geumyoung00 geumyoung00 commented Aug 6, 2025

🚀 풀 리퀘스트 제안

📋 작업 내용

수정한 내용이나 추가한 기능에 대해 자세히 설명해 주세요.

🔧 변경 사항

  • 📃 README.md
  • 📦 package.json
  • 🔥 파일 삭제
  • 🧹 그 외 ex) .gitignore 등

주요 변경 사항을 요약해 주세요.

📸 스크린샷 (선택 사항)

수정된 화면 또는 기능을 시연할 수 있는 스크린샷을 첨부해 주세요.

📄 기타

추가적으로 전달하고 싶은 내용이나 특별한 요구 사항이 있으면 작성해 주세요.

Sourcery 요약

채팅 및 메인 레이아웃 전반에 걸쳐 사용되지 않는 헤더 기능을 숨기고, 일관된 헤더 스타일링을 위해 제목을 중앙 정렬합니다.

버그 수정:

  • 채팅 헤더에서 Bell 및 ButtonMore 아이콘을 주석 처리합니다.
  • 메인 헤더에서 BellRing 아이콘을 제거합니다.
  • 채팅 목록 카드에서 ButtonChatMore 컴포넌트를 주석 처리합니다.

개선 사항:

  • 일관된 레이아웃을 위해 절대 위치 지정을 사용하여 헤더 제목을 중앙 정렬합니다.
Original summary in English

Summary by Sourcery

Hide unused header features across the chat and main layouts and center-align titles for consistent header styling.

Bug Fixes:

  • Comment out the Bell and ButtonMore icons in the chat header
  • Remove the BellRing icon from the main header
  • Comment out the ButtonChatMore component in the chat list card

Enhancements:

  • Center-align header titles using absolute positioning for consistent layout

@geumyoung00 geumyoung00 requested a review from dolmeengii August 6, 2025 05:19
@sourcery-ai
Copy link
Contributor

sourcery-ai bot commented Aug 6, 2025

검토자 가이드

이 PR은 채팅, 메인, 채팅 목록 카드 전반에 걸쳐 사용되지 않는 헤더 액션 컴포넌트를 주석 처리하여 숨기고, 절대 위치 지정 및 상대 컨테이너 래퍼를 통해 제목을 중앙 정렬하도록 채팅 헤더 레이아웃을 개선합니다.

파일별 변경 사항

변경 사항 세부 정보 파일
사용되지 않는 헤더 액션 컴포넌트 숨기기
  • 채팅 헤더의 벨 아이콘 주석 처리
  • 채팅 헤더의 ButtonMore 및 해당 대체 주석 처리
  • 메인 헤더에서 BellRing 아이콘 제거
  • 채팅 목록 카드의 ButtonChatMore 주석 처리
apps/web/src/components/layout/header/chat.tsx
apps/web/src/components/layout/header/main.tsx
apps/web/src/components/chat/chat-list-card.tsx
헤더 제목 중앙 정렬
  • 헤더

    요소를 translate-x 중앙 정렬과 함께 절대 위치로 변환

  • 헤더 컨테이너에 상대 위치 지정 클래스 추가
apps/web/src/components/layout/header/chat.tsx

팁 및 명령

Sourcery와 상호 작용하기

  • 새로운 검토 트리거: 풀 리퀘스트에 @sourcery-ai review라고 댓글을 남기세요.
  • 논의 계속: Sourcery의 검토 댓글에 직접 답글을 남기세요.
  • 검토 댓글에서 GitHub 이슈 생성: Sourcery에게 검토 댓글에 답글을 달아 이슈를 생성하도록 요청할 수 있습니다. 또는 검토 댓글에 @sourcery-ai issue라고 답글을 달아 이슈를 생성할 수도 있습니다.
  • 풀 리퀘스트 제목 생성: 풀 리퀘스트 제목 아무 곳에나 @sourcery-ai를 작성하여 언제든지 제목을 생성할 수 있습니다. 또는 풀 리퀘스트에 @sourcery-ai title이라고 댓글을 달아 언제든지 제목을 (재)생성할 수 있습니다.
  • 풀 리퀘스트 요약 생성: 풀 리퀘스트 본문 아무 곳에나 @sourcery-ai summary를 작성하여 원하는 위치에 언제든지 PR 요약을 생성할 수 있습니다. 또는 풀 리퀘스트에 @sourcery-ai summary라고 댓글을 달아 언제든지 요약을 (재)생성할 수 있습니다.
  • 검토자 가이드 생성: 풀 리퀘스트에 @sourcery-ai guide라고 댓글을 달아 언제든지 검토자 가이드를 (재)생성할 수 있습니다.
  • 모든 Sourcery 댓글 해결: 풀 리퀘스트에 @sourcery-ai resolve라고 댓글을 달아 모든 Sourcery 댓글을 해결하세요. 모든 댓글을 이미 처리하여 더 이상 보고 싶지 않을 때 유용합니다.
  • 모든 Sourcery 검토 해제: 풀 리퀘스트에 @sourcery-ai dismiss라고 댓글을 달아 기존의 모든 Sourcery 검토를 해제하세요. 새로운 검토로 처음부터 다시 시작하고 싶을 때 특히 유용합니다. 새로운 검토를 트리거하려면 @sourcery-ai review라고 댓글을 다는 것을 잊지 마세요!

경험 사용자 지정

대시보드에 접속하여 다음을 수행하세요:

  • Sourcery가 생성한 풀 리퀘스트 요약, 검토자 가이드 등 검토 기능을 활성화 또는 비활성화합니다.
  • 검토 언어를 변경합니다.
  • 사용자 지정 검토 지침을 추가, 제거 또는 편집합니다.
  • 다른 검토 설정을 조정합니다.

도움말

Original review guide in English

Reviewer's Guide

The PR conceals unused header action components across chat, main, and chat-list-card by commenting them out, and refines the chat header layout to center-align the title through absolute positioning and a relative container wrapper.

File-Level Changes

Change Details Files
Hide unused header action components
  • Comment out Bell icon in chat header
  • Comment out ButtonMore in chat header and its fallback
  • Remove BellRing icon from main header
  • Comment out ButtonChatMore in chat list card
apps/web/src/components/layout/header/chat.tsx
apps/web/src/components/layout/header/main.tsx
apps/web/src/components/chat/chat-list-card.tsx
Center align header titles
  • Convert header

    elements to absolute position with translate-x centering

  • Add relative positioning class to header container
apps/web/src/components/layout/header/chat.tsx

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@dolmeengii dolmeengii merged commit 4ffba38 into Kernel360:develop Aug 6, 2025
2 checks passed
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

안녕하세요 @geumyoung00님 - 변경사항을 검토했는데 아주 좋습니다!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
제가 더 유용해질 수 있도록 도와주세요! 각 댓글에 👍 또는 👎를 클릭해 주시면, 피드백을 활용하여 검토를 개선하겠습니다.
Original comment in English

Hey @geumyoung00 - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants