Feat(client): 메모 검색 모달 구현 - #288
Open
jyeon03 wants to merge 3 commits into
Open
Conversation
jyeon03
requested review from
jm8468,
jogpfls and
twossu
and removed request for
a team
August 18, 2026 07:56
🎨 Storybook 배포 완료PR 작성자: @jyeon03 |
twossu
approved these changes
Aug 22, 2026
twossu
left a comment
Member
There was a problem hiding this comment.
저는 empty뷰를 정말 새까맣게 잊고있었네요
MemoSearchModal props 구조가 검색 전/후 상태를 표현하기에 적절한지 확인 부탁드립니다.
recentMemos와 searchResultMemos를 나눠 받는 부분에서 감탄하고 갑니다. 저는 처음엔 어차피 화면에 보이는 건 하나의 리스트니까 props도 하나로 합치면 되지 않을까 싶었는데 그렇게 합치면 어떤 리스트를 보여줄지 판단하는 로직을 상위 컴포넌트에서 처리해야 해서 오히려 좋지 않을 것 같아요. 상태를 이렇게 분리해서 받는 방식이 이번에 처음 봤는데 감탄하고갑니다.
| gap: '0.4rem', | ||
| flex: 1, | ||
| minHeight: 0, | ||
| padding: '1.8rem 0.8rem 1.8rem 2rem', |
Member
There was a problem hiding this comment.
Suggested change
| padding: '1.8rem 0.8rem 1.8rem 2rem', | |
| padding: '1.8rem 2rem' |
body에 상하좌우 동일하게 패딩 넣고 list에서 패딩 설정하지 않아도 될 것 같은데 나눠서 스타일 적용한것이 궁금합니da🎃
Comment on lines
+37
to
+39
| const sectionTitle = isSearchResult | ||
| ? getSearchResultSectionTitle(resultCount) | ||
| : '최근 열람한 메모'; |
Member
There was a problem hiding this comment.
isSearchResult, resultCount는 sectionTitle에만 쓰이니 인라인으로 처리해도 좋을 것 같아엽떡
⚰️🦇🧛🏼
Suggested change
| const sectionTitle = isSearchResult | |
| ? getSearchResultSectionTitle(resultCount) | |
| : '최근 열람한 메모'; | |
| const sectionTitle = searchResultMemos | |
| ? getSearchResultSectionTitle(searchResultMemos.length) | |
| : '최근 열람한 메모'; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌 Summary
feat/memo-search-list-item/#271위에서 작업한 Stack PR입니다. 검색 모달 내부에서MemoSearchListItem컴포넌트를 사용해야 해서, 해당 브랜치의 변경사항을 기반으로 작업했습니다.📚 Tasks
SearchBar를memo-search-modalfeature 하위로 이동MemoSearchModal컴포넌트 구현🔍 Describe
1. Stack PR 구조
feat/tag-component-redesign/#241 └── feat/memo-search-list-item/#271 └── feat/search-modal/#2852. 폴더 구조
SearchBar,MemoSearchListItem은 검색 모달 내부에서 조합되는 세부 UI라components/폴더 아래로 정리했습니다.최상위 조립 컴포넌트인
MemoSearchModal은 폴더 바로 아래에 두고, 외부에서는index.ts를 통해 import할 수 있도록 했습니다.features/ └── memo-search-modal/ ├── index.ts ├── memo-search-modal.tsx ├── memo-search-modal.css.ts ├── types.ts └── components/ ├── search-bar/ │ ├── search-bar.tsx │ ├── search-bar.css.ts │ └── search-bar.test.tsx └── memo-search-list-item/ ├── memo-search-list-item.tsx └── memo-search-list-item.css.ts3. 검색 전 / 검색 후 상태 처리
recentMemossearchResultMemossearchResultMemos가undefined이면 검색 전 상태로 보고, 배열이면 검색 완료 상태로 판단하도록 했습니다.4. 검색 결과 개수 계산
5. Props 역할
open: 모달 열림 여부searchValue: 검색바 입력값recentMemos: 검색 전 보여줄 최근 열람 메모 목록searchResultMemos: 검색 완료 후 보여줄 검색 결과 목록onOpenChange: 모달 열림/닫힘 상태 변경onChangeSearchValue: 검색어 입력값 변경onSearch: Enter로 검색을 확정했을 때 호출onClickMemo: 메모 아이템 클릭 시 호출6. Empty 상태
// TODO: empty view 컴포넌트 디자인 확정되는 대로 교체 예정검색 결과 없음 / 최근 메모 없음 상태는 다른 모달에서도 공통으로 쓰일 수 있는 UI라, 추후 별도 shared 컴포넌트 작업에서 교체할 예정입니다.
👀 To Reviewer
MemoSearchModalprops 구조가 검색 전/후 상태를 표현하기에 적절한지 확인 부탁드립니다.searchResultMemos === undefined를 검색 전 상태로 판단하는 방식이 괜찮은지 확인 부탁드립니다.📸 Screenshot
2026-08-18.4.43.24.mov