Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[REFACTOR] 검색 페이지 코드 리팩터링 #121

Merged
merged 20 commits into from
Jun 19, 2024
Merged

[REFACTOR] 검색 페이지 코드 리팩터링 #121

merged 20 commits into from
Jun 19, 2024

Conversation

rbgksqkr
Copy link
Collaborator

@rbgksqkr rbgksqkr commented Jun 19, 2024 •
edited
Loading

📌 PR 내용

  • 스타일 코드 디렉토리 및 파일 분리
  • 삼항 연산자 코드 가독성 개선
  • useSuspenseQuery와 React.Suspense를 활용하여 선언적 로딩 UI 처리
  • 검색어(searchKeyword)가 없을 때 요청 처리
  • 반응형 디자인 일부 수정 (모바일 기준 576px -> 768px)
  • 500px 이하 시 꺠지는 디자인 문제 수정
  • 컴포넌트 내의 혼재되어 잇는 비즈니스 로직 커스텀 훅으로 분리
  • 자동완성창을 absolute로 설정 (레이아웃이 움직이는 것보다 가리는 게 낫다고 판단)

📝 코드 요약

-'''search'''

💌 해결 이슈

📸 스크린샷 (선택)

1024px

1024px

768px

768px

400px

400px

gominzip and others added 20 commits June 9, 2024 21:19
@gominzip
feat : favicon 수정
1a35998
@gominzip
chore : 로그인 안내문구 변경
f7b8d60
@rbgksqkr
chore: 스타일 코드 디렉토리 및 파일 분리(#120)
96ef577
@rbgksqkr
refactor: 삼항연산자로 복잡한 JSX 반환하는 컴포넌트 구조 개선(#120)
9f50ac5
@rbgksqkr
fix: 같은 검색결과 반환값을 캐싱하도록 queryKey 수정(#120)
4c5934d
@rbgksqkr
chore: 사용처에서 네이밍을 변경하도록 수정(#120)
5d2fe97
@rbgksqkr
refactor: 불필요한 리렌더링 개선(#120)
881e47c
@rbgksqkr
chore: react-query 중복 설치 제거(#120)
29e86cf
@rbgksqkr
refactor: useSuspenseQuery와 Suspense를 활용하여 로딩 UI 처리(#120)
62fdf91
@rbgksqkr
fix: searchKeyword가 빈 문자열일 때 API 호출을 막기 위해 Promise 반환(#120)
6e0b709
@rbgksqkr
refactor: 자동완성창 외부 클릭시 창닫기 로직을 id 대신 ref 를 활용하여 수정(#120)
1575188
@rbgksqkr
refactor: 자동완성창 관리하는 로직을 useFocusSearchInput 커스텀 훅으로 분리하고 불필요한 form 형…
83088d2 
…식 제거(#120)
@rbgksqkr
refactor: 불필요한 file deps를 제거하고, params를 각각의 컴포넌트에서 읽도록 수정(#120)
057f234
@rbgksqkr
feat: 검색 API 호출 시, 검색 페이지 입력값과 Navbar 입력값 동기화하기(#120)
e8dfa5e
@rbgksqkr
chore: form 태그 제거로 인해 파일명 변경(#120)
bfd5361
@rbgksqkr
refactor: 검색어와 100% 일치하는 데이터가 있는 경우 라우팅하는 커스텀 훅 분리(#120)
98d45cb
@rbgksqkr
chore: 파일명 수정(#120)
a897c57
@rbgksqkr
fix: 문서 페이지로 이동해도 Navbar 입력값 유지(#120)
b7d9a4e
@rbgksqkr
design: 검색 화면 반응형 디자인 수정(#120)
46b8946
@rbgksqkr
chore: 중복되는 css 제거(#120)
2ff5eea
@vercel Vercel
Copy link

vercel bot commented Jun 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
kiwi-client ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 19, 2024 9:44am

@rbgksqkr rbgksqkr merged commit 10fadf5 into develop Jun 19, 2024
2 checks passed
@rbgksqkr rbgksqkr self-assigned this Jun 19, 2024
@rbgksqkr rbgksqkr added DESIGN css 수정 REFACTOR 리팩토링 labels Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@rbgksqkr rbgksqkr

Labels
DESIGN css 수정 REFACTOR 리팩토링
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rbgksqkr @gominzip