Skip to content

[♻️ Refactor] 이미지업로드 아이콘 변경#203

Merged
didwjdgns99 merged 9 commits intomainfrom
Refactor/imageupload_icon
Jan 18, 2026
Merged

[♻️ Refactor] 이미지업로드 아이콘 변경#203
didwjdgns99 merged 9 commits intomainfrom
Refactor/imageupload_icon

Conversation

@didwjdgns99
Copy link
Collaborator

1. 코드 & 기능

  • 기능 정상 동작
  • 콘솔 에러 없음

🔗 이슈 번호

  • close #이슈번호

🎯 작업한 내용

💡 리뷰 요청 / 코멘트

🫧 참고사항

📸 스크린샷 (선택사항)

@vercel
Copy link

vercel bot commented Jan 17, 2026

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

Project Deployment Review Updated (UTC)
global-nomad-team6 Ready Ready Preview, Comment Jan 18, 2026 6:41am

@gemini-code-assist
Copy link
Contributor

Summary of Changes

Hello @didwjdgns99, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 이미지 업로드 기능의 아이콘을 변경하여 리팩토링하는 것을 주요 목표로 합니다. 새로운 FallbackImage SVG 아이콘을 도입하고, ImageUpload 컴포넌트가 기존의 PasswordHidden 아이콘 대신 이 새로운 아이콘을 사용하도록 업데이트했습니다. 또한, ReservationPage 컴포넌트에서 사소한 CSS 조정이 이루어졌습니다.

Highlights

  • 아이콘 변경: ImageUpload 컴포넌트에서 사용되던 PasswordHidden 아이콘이 새로운 FallbackImage 아이콘으로 교체되었습니다.
  • 새 아이콘 추가: fallback_image.svg라는 새로운 SVG 아이콘이 에셋에 추가되었습니다.
  • 임포트 업데이트: ImageUpload.tsx 컴포넌트가 이제 새로운 FallbackImage 아이콘을 임포트합니다.
  • CSS 클래스 제거: ReservationPage.tsx 파일에서 특정 div 요소의 overflow-x-auto CSS 클래스가 제거되었습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

이미지 업로드 아이콘을 변경하는 리팩토링이 포함된 PR이네요. 아이콘 변경은 좋아 보입니다. 다만, 리뷰 과정에서 몇 가지 개선점을 발견했습니다. ImageUpload 컴포넌트에서 사용하지 않는 import를 제거하고, ReservationPage에서 발생할 수 있는 UI 버그를 수정하는 것을 제안합니다. 자세한 내용은 각 파일의 주석을 참고해주세요.

</div>
{hasAnyReservation && (
<div className='scrollbar-hide mr-6 flex flex-nowrap gap-2 overflow-x-auto pb-[13px] md:pb-7.5'>
<div className='scrollbar-hide mr-6 flex flex-nowrap gap-2 pb-[13px] md:pb-7.5'>
Copy link
Contributor

Choose a reason for hiding this comment

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

high

overflow-x-auto 클래스가 제거되었습니다. 필터 버튼들이 한 줄에 표시되도록 flex-nowrap이 적용되어 있는데, 화면 너비가 좁을 경우 일부 버튼이 잘려서 보이지 않게 될 수 있습니다. 사용자가 모든 필터 옵션에 접근할 수 있도록 overflow-x-auto를 다시 추가하여 가로 스크롤이 가능하도록 하는 것을 권장합니다.

Suggested change
<div className='scrollbar-hide mr-6 flex flex-nowrap gap-2 pb-[13px] md:pb-7.5'>
<div className='scrollbar-hide mr-6 flex flex-nowrap gap-2 overflow-x-auto pb-[13px] md:pb-7.5'>

@@ -1,4 +1,4 @@
import { Delete, PasswordHidden } from '@/assets/icons';
import { Delete, FallbackImage, PasswordHidden } from '@/assets/icons';
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

PasswordHidden 아이콘이 FallbackImage로 대체되면서 더 이상 사용되지 않는 것 같습니다. 불필요한 import를 제거하여 코드를 더 깔끔하게 유지하는 것이 좋습니다.

Suggested change
import { Delete, FallbackImage, PasswordHidden } from '@/assets/icons';
import { Delete, FallbackImage } from '@/assets/icons';

Copy link
Contributor

@le2yunji le2yunji left a comment

Choose a reason for hiding this comment

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

src/components/common/image-upload/ImageUpload.tsx(1,33): error TS6133: 'PasswordHidden' is declared but its value is never read.

정훈님 해당 에러때문에 vercel 오류가 발생하고 있습니다 !

@didwjdgns99 didwjdgns99 merged commit 4d54a48 into main Jan 18, 2026
2 checks passed
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.

4 participants