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/#289: 공지사항, FAQ 페이지에 UpperLayout 합성 컴포넌트 적용 & 팁 페이지 이미지 수정 #292

Merged
merged 19 commits into from
Dec 3, 2023
Merged

refactor/#289: 공지사항, FAQ 페이지에 UpperLayout 합성 컴포넌트 적용 & 팁 페이지 이미지 수정 #292

merged 19 commits into from
Dec 3, 2023

Conversation

hwinkr
Copy link
Collaborator

@hwinkr hwinkr commented Dec 1, 2023

🤠 개요

  • InformUpperLayout 검색 바 구현추가하기
  • 홈 페이지 카드 아이콘(학교, 학과) 구분
  • FAQ 페이지 아이콘 방향 변경
  • 팁 페이지 이미지 크기 맞추기

💫 설명

  • 공지사항, FAQ 페이지에 InformUpperLayout 합성 컴포넌트 적용했어요.
  • UpperLayout 컴포넌트에 InformSearchForm 컴포넌트를 추가 구현 했어요.
  • 꿀팁 페이지에서 사용되는 백경이 이미지들이 크기가 맞지 않아 이미지가 깨지는 문제가 발생해 크기를 통일 시켰어요.
  • FAQ 페이지에서 화살표 아이콘 애니메이션을 변경했어요

📷 스크린샷 (Optional)

hwinkr added 16 commits November 30, 2023 16:50
@hwinkr
chore(delete unused component): 사용하지 않는 컴포넌트 삭제
80910bf
@hwinkr
chore(modify image demension): 팁 페이지에서 사용할 백경이 png 이미지 변경
4059469 
- 기존에 사용했던 백경이 이미지들은 크기가 일치하지 않아서 이미지가 늘어지는 문제가 발생
- 이미지를 2048 x 2048로 변경
- 백경이 손하트 이미지 추가
@hwinkr
chore(modify image demension): 팁 페이지에서 사용할 백경이 webp 이미지 변경
888288f 
- 기존에 사용했던 백경이 이미지들은 크기가 일치하지 않아서 이미지가 늘어지는 문제가 발생
- 이미지를 2048 x 2048로 변경
- 백경이 손하트 이미지 추가
@hwinkr
chore(add icon): 아래방향 화살표, 학교 빌딩 아이콘 추가
d81b08c
@hwinkr
refactor(modify icon type): 컴포넌트에서 사용되는 아이콘 타입 변경
69de644
@hwinkr
refactor(edit test code): 컴포넌트에서 사용하는 아이콘이 변경되어, 테스트 코드 수정
de84e25
@hwinkr
refactor(modify icon animation): 사용자가 FAQ를 클릭할 경우 적용됐던 화살표 아이콘 애니메이션 …
53688cc 
…방향 변경
@hwinkr
feat(add InformSearchForm): 합성 컴포넌트 구성에 InformSearchForm 컴포넌트 추가
1f6530b
@hwinkr
feat(InformSearchForm): 정보를 검색할 수 있는 검색바 컴포넌트 구현
dd24137 
- 검색 키워드에 대한 유효성 검사는 불필요하기 때문에 useRef 훅을 사용해서 구현
- 아무 키워드를 입력하지 않을 경우 토스트 메세지를 렌더링
- 키워드 입력을 받아서 search/?q=<keyword>로 이동시키는 책임을 가짐
@hwinkr
refactor(add InformSearchForm): children 컴포넌트들의 타입을 확인하는 함수에 검색 바 컴포넌…
4c454ef 
…트 추가
@hwinkr
refactor(remove boundaryLine): 공지사항 리스트 컴포넌트에서 BoundaryLine 컴포넌트 삭제
3ac931f 
- 공지사항 리스트만 보여주면 되므로 BoundaryLine 까지 구현할 책임은 없다고 판단
@hwinkr
chore(modify image import path): 팁 페이지에서 사용할 이미지 import 경로 수정
5f90f7e
@hwinkr
refactor(use compound component): 공지사항 컨테이너 컴포넌트에서 상단 레이아웃 합성 컴포넌트를 적용
5289959 
- 상단 레이아웃, 공지사항 리스트를 구분하는 BoundaryLine 추가
@hwinkr
chore(delete unused component): 사용하지 않는 컴포넌트 export 경로에서 제거
4a4b806
@hwinkr
chore(edit usage icon): 학교, 학과 공지사항 카드 컴포넌트에서 사용되는 아이콘 구분
a1cce3d
@hwinkr
refactor(use compound component): FAQ 페이지에 상단 레이아웃 합성 컴포넌트 적용
4f6c3b0 
- 상단 레이아웃, FAQ 리스트를 구분하는 BoundaryLine을 div -> hr로 변경
@hwinkr hwinkr requested a review from pp449 December 1, 2023 07:08
@hwinkr hwinkr self-assigned this Dec 1, 2023
pp449
pp449 requested changes Dec 1, 2023
View reviewed changes
Copy link
Member

@pp449 pp449 left a comment

Choose a reason for hiding this comment

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

image

검은선과 회색선의 두께가 안맞아서 그런지 첫번째 질문이 정확한 중앙이 아닌거 같아요 요거만 수정하면 좋을거 같아요!

@hwinkr
Copy link
Collaborator Author

hwinkr commented Dec 1, 2023

image 검은선과 회색선의 두께가 안맞아서 그런지 첫번째 질문이 정확한 중앙이 아닌거 같아요 요거만 수정하면 좋을거 같아요

수정 완료~!

pp449
pp449 approved these changes Dec 2, 2023
View reviewed changes
Copy link
Member

@pp449 pp449 left a comment

Choose a reason for hiding this comment

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

GOOD~~

@hwinkr hwinkr merged commit 6d43064 into dev Dec 3, 2023
1 check passed
@hwinkr hwinkr deleted the refactor/#289 branch December 3, 2023 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pp449 pp449 pp449 approved these changes

Assignees

@hwinkr hwinkr

Labels
🧑‍🔧 refactor 💇‍♂️ style
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Refactor: 공지사항 페이지에 UpperLayout 합성 컴포넌트를 적용하도록 수정
2 participants
@hwinkr @pp449