Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Summary of ChangesHello @dodaaaaam, 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은 '사진수다' 기능의 4차 QA 피드백을 반영하기 위한 리팩토링으로, 사용자 인터페이스 및 경험 개선에 중점을 둡니다. 주요 변경 사항으로는 검색 기능에 임시 필터 선택 메커니즘을 도입하여 사용성을 높이고, 모바일 기기에서 안전 영역 패딩을 조정하여 UI 요소가 올바르게 표시되도록 하며, 다양한 사진 관련 페이지에서 레이아웃과 간격을 세밀하게 조정하는 작업이 포함됩니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Activity
Using Gemini Code AssistThe 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
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 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
|
There was a problem hiding this comment.
Code Review
사진수다 4차 QA 반영을 위한 리팩토링 작업이네요. 전반적으로 UI/UX를 개선하기 위한 변경 사항들이 인상적입니다. 특히 iOS 기기의 safe area를 고려한 레이아웃 조정과 검색 필터 기능의 사용자 경험 개선이 돋보입니다. 코드 리뷰 중 몇 가지 추가 개선점을 발견하여 코멘트를 남겼습니다. 주로 검색 페이지의 고정 헤더 도입에 따른 레이아웃 문제와 스타일 클래스명 오타에 관한 내용이니 확인 후 반영해주시면 좋겠습니다.
| <div className="fixed top-0 right-0 left-0 z-40 mx-4 bg-neutral-900"> | ||
| {/* SearchBar */} | ||
| <div className="mt-3 mb-5"> | ||
| <SearchBar {...searchBarProps} /> | ||
| </div> | ||
|
|
||
| {/* 검색 결과 n개 + 필터링 (result 모드일 때만 고정) */} | ||
| {mode === "result" && ( | ||
| <div className="mb-4 flex flex-col gap-4"> | ||
| <div className="flex items-center justify-between"> | ||
| <div className="flex items-center gap-2"> | ||
| <h2 className="text-[1rem] leading-[155%] font-semibold tracking-[-0.02em] text-neutral-100"> | ||
| 검색 결과 | ||
| </h2> | ||
| <p className="text-[1rem] font-light text-neutral-100"> | ||
| {totalCount}개 | ||
| </p> | ||
| </div> | ||
| <button | ||
| type="button" | ||
| onClick={() => { | ||
| setBottomSheetOpen(true); | ||
| }} | ||
| className="flex items-center gap-[6px] text-[0.875rem] leading-[155%] font-normal tracking-[-0.02em] text-neutral-400" | ||
| > | ||
| <span>{FILTER_LABEL[filter]}</span> | ||
| <ChevronLeftIcon className="h-4 w-4 rotate-[-90deg] text-neutral-200" /> | ||
| </button> | ||
| </div> | ||
| </div> | ||
| )} | ||
| </div> |
There was a problem hiding this comment.
검색 바와 필터링 UI를 상단에 고정하여 사용성을 개선한 점이 좋습니다. 다만 이 변경으로 인해 몇 가지 레이아웃 문제가 발생한 것으로 보입니다.
-
UI 중복 렌더링:
mode === 'result'일 때 '검색 결과' 헤더가 현재 이 고정 컴포넌트와renderResult함수(301-320 라인) 양쪽에서 렌더링되어 화면에 중복으로 표시됩니다.renderResult함수 내에서 해당 부분을 제거해야 합니다. -
콘텐츠 가려짐 문제:
renderRecent에mt-20을 추가하여 콘텐츠가 가려지는 문제를 해결하셨지만,renderRelated와renderResult에서는 여전히 콘텐츠가 고정 헤더에 의해 가려집니다. 이 두 경우에도 적절한margin-top을 추가하여 레이아웃을 수정해야 합니다. 특히result모드는 헤더의 높이가 더 크므로 더 큰 마진이 필요합니다.
src/pages/photoFeed/PostPage.tsx
Outdated
| <div className="flex items-center gap-2"> | ||
| <HomeIcon className="h-4 w-4 font-semibold" /> | ||
| <p className="text-[1rem] font-semibold text-neutral-200"> | ||
| <p className="font-Medium text-[1rem] text-neutral-200"> |
There was a problem hiding this comment.
SHOULD: 슬슬 이쪽에 의존성이 너무 많아지는 거 같아서.. 몇몇 기능은 분리해도 좋을 거 같습니당
MlNTYS
left a comment
There was a problem hiding this comment.
수고하셨습니다! 추후에 SearchPage 쪽 hook 분리 고려 부탁드려요!
🔀 Pull Request Title
사진수다 4차 QA 반영
📌 PR 설명
이번 PR에서 어떤 작업을 했는지 요약해주세요.