Skip to content

Comments

[Feat/search] 로고 변경#48

Closed
fullkeem wants to merge 8 commits intodevfrom
feat/search
Closed

[Feat/search] 로고 변경#48
fullkeem wants to merge 8 commits intodevfrom
feat/search

Conversation

@fullkeem
Copy link
Collaborator

📝 PR 제목

  • 로고 변경

📌 변경 사항

  • 로고 변경
  • 검색바를 네브바에 추가
  • sortSelect UI 변경

🚀 작업 목적

  • 사용자가 이메일과 비밀번호로 로그인할 수 있도록 합니다.

🔄 관련 이슈


📸 변경 전/후 화면 (Optional)

sort select 변경 navBar 변경
image image

✅ 체크리스트

  • 코드가 정상적으로 동작하는지 테스트했습니다.
  • 코드에 불필요한 주석이나 로그를 제거했습니다.
  • ESLintPrettier를 실행했습니다.
  • 관련된 문서를 업데이트했습니다.

🧐 중점적으로 리뷰받고 싶은 부분

  • 코드 로직에 개선할 부분이 있는지 확인해주세요.
  • 에러 핸들링 방식이 적절한지 봐주세요.

🤝 PR 리뷰어에게 요청사항

  • 코드 스타일 관련 피드백 환영합니다.
  • 궁금한 부분이 있으면 언제든 질문해주세요. 😊

commit 38ba848
Merge: 00190c0 8525685
Author: shchoon <87121448+shchoon@users.noreply.github.com>
Date:   Fri Jan 24 17:06:45 2025 +0900

    Merge pull request #47 from booksaetong/feat/bookshelf1

    [Feat/modal] 모달 컴포넌트 수정

commit 8525685
Author: SeanKim05 <dbgus20609@gmail.com>
Date:   Fri Jan 24 14:20:55 2025 +0900

    Fix: 모달 컴포넌트 수정

    - 모달 백그라운드 스크롤 방지 로직 추가
    - 모달 컴포넌트 구조 수정
    - 북마크 조건부 표시 : 서재 페이지에서는 북마크 숨김

commit 00190c0
Merge: 6931186 97b4da7
Author: shchoon <87121448+shchoon@users.noreply.github.com>
Date:   Fri Jan 24 10:09:58 2025 +0900

    Merge pull request #46 from booksaetong/feat/search

    [Feat/search] 모달 북마크 추가
- 로고 변경
- 로고 크기 및 서재 아이콘 크기 변경
- 검색 결과 제목 폰트 변경
- SearchResult 요소 key 값의 isbn 중 isbn-13으로 변경
- 드롭다운 form 내에 button의 type="button" 지정
@fullkeem fullkeem closed this Jan 26, 2025
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.

1 participant