Skip to content

[#138] refactor(container): Bevel 스크롤 컨테이너 분리/적용 - #141

Merged
sageherb merged 6 commits into
devfrom
refactor/bevel-wrapper-container
Nov 19, 2025
Merged

sageherb merged 6 commits into
devfrom
refactor/bevel-wrapper-container

Conversation

@RumPumpumpum

@RumPumpumpum RumPumpumpum commented Nov 19, 2025 •

Copy link
Copy Markdown
Collaborator

📖 개요

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

✅ 관련 이슈

🛠️ 상세 작업 내용

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

📸 스크린샷

image image image image

⚠️ 주의 사항

👥 리뷰 확인 사항

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

@vercel

vercel Bot commented Nov 19, 2025 •

Copy link
Copy Markdown

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

Copilot AI left a comment

Copy link
Copy Markdown

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.

Comment thread src/features/friends/list/List.tsx
Comment thread src/components/Container/BevelScrollContainer.tsx Outdated
Comment thread src/features/minihome/post/list/PostList.tsx
Comment thread src/features/minihome/post/list/PostList.tsx
Comment thread src/features/minihome/guestbook/GuestBook.tsx
Comment thread src/features/friends/search/Search.tsx
Comment thread src/features/friends/request/Request.tsx

@sageherb sageherb left a comment

Copy link
Copy Markdown
Collaborator

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

This branch was successfully deployed

1 active deployment
Preview — f6a1308b Deployed Nov 19, 2025 by vercel[bot]
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