Skip to content

Conversation

@RumPumpumpum
Copy link
Collaborator

@RumPumpumpum RumPumpumpum commented Nov 19, 2025

📖 개요

  • 중복되던 베벨+스크롤 래퍼를 BevelScrollContainer로 분리하고 갤러리/메모/방명록/친구창 리스트 영역에 공통 적용했습니다.

✅ 관련 이슈

🛠️ 상세 작업 내용

  • src/components/Container/BevelScrollContainer.tsx 추가
  • 갤러리 리스트·업로드 등 미니홈 컴포넌트에 컨테이너 적용
  • 메모 탭(게시글 목록/상세)에 컨테이너 적용
  • 방명록 리스트에 컨테이너 적용
  • 친구 Search/Request/List 화면에 컨테이너 적용 및 import 정리

📸 스크린샷

image image image image

⚠️ 주의 사항

👥 리뷰 확인 사항

공통적으로 갤러리의 스타일인 p-4를 적용했습니다.

@vercel
Copy link

vercel bot commented Nov 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
backspace Ready Ready Preview Comment Nov 19, 2025 2:28pm

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR extracts a common bevel + scroll wrapper pattern into a reusable BevelScrollContainer component and applies it across gallery, memo (post list), guestbook, and friend management screens to reduce code duplication.

Key changes:

  • New BevelScrollContainer component that encapsulates bevel styling with scrollable content area
  • Standardized padding to p-4 across all implementations (previously varied between p-3, p-3 pr-0, pl-4, etc.)
  • Simplified markup by replacing repetitive two-div patterns with a single component

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
src/components/Container/BevelScrollContainer.tsx New reusable container component with bevel border and scrollable content area
src/features/minihome/post/list/PostList.tsx Applied BevelScrollContainer to post list, replacing previous bevel+scroll wrapper
src/features/minihome/guestbook/GuestBook.tsx Applied BevelScrollContainer to guestbook list with flex wrapper
src/features/minihome/gallery/GalleryList.tsx Applied BevelScrollContainer to gallery grid view
src/features/friends/search/Search.tsx Applied BevelScrollContainer to friend search results with flex wrapper
src/features/friends/request/Request.tsx Applied BevelScrollContainer to friend request list with flex wrapper
src/features/friends/list/List.tsx Applied BevelScrollContainer to friend list with flex wrapper

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@sageherb sageherb left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@sageherb sageherb merged commit ca6c513 into dev Nov 19, 2025
2 checks passed
@sageherb sageherb deleted the refactor/bevel-wrapper-container branch November 19, 2025 14:28
@RumPumpumpum RumPumpumpum changed the title [#138] Bevel 스크롤 컨테이너 분리/적용 [#138] refactor(container): Bevel 스크롤 컨테이너 분리/적용 Nov 20, 2025
@sageherb sageherb added the refactor 코드 리팩토링 / 구조 개선 label Nov 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor 코드 리팩토링 / 구조 개선

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task] 베벨 테두리 래퍼와 스크롤바를 컨테이너로 분리 및 교체

3 participants