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.

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

[#35] 질문 조회 페이지 #39

Merged
merged 8 commits into from
Aug 23, 2023
Merged

[#35] 질문 조회 페이지 #39

merged 8 commits into from
Aug 23, 2023

Conversation

intersoom
Copy link
Member

Summary

질문 조회 페이지 제작

Description

Details

Routing

  • [pid].tsx라는 파일을 만들어서 동적 라우팅이 가능하게 하였습니다.

Search

  • 필터 버튼 클릭 시, 드롭다운이 열리게 하였습니다.
    • 재사용이 가능하게끔, Search 컴포넌트의 props로 variant 옵션을 up | down으로 받아서 열리는 방향을 선택 가능하게끔 하였습니다. (홈화면은 위로 열림)
    • input과 버튼은 기존에 있는 컴포넌트 사용하였습니다.

Question

  • Tag 컴포넌트 사용하여서 UI 맞추어 제작
  • UI 맞춰서 제작

MyAnswer

  • 마크다운 재사용

Key Changes

  • search 관련 파일 일부를 reusable 폴더로 이전했습니다.
  • searchInput border 옵션을 추가하여 재사용성을 올렸습니다.
    • 기본은 true로 설정하여서 기존의 컴포넌트들이 수정할 필요는 없게 하였습니다.

To Reviewer

  • 리뷰어에게 전달할 내용에 대해서 작성해주세요

After Work

  • 질문 작성 페이지에서도 사용한 로직을 해당 페이지에서도 동일하게 사용하여서 이를 묶을 수 있는 방법도 생각해봐도 좋을 것 같습니다!
    • 외부 영역 터치시 드롭다운 닫히는 것
  • 답변하기 버튼 비활성화
  • 답변 시, 답변창 비활성화
  • API 연결
  • 질문 번호에 따라서 pid 라우팅
  • Tag 컴포넌트 overflow 고려 (기획 / 디자이너님과 논의)

@intersoom intersoom self-assigned this Aug 15, 2023
@intersoom intersoom requested a review from seocylucky August 15, 2023 17:08

const Container = styled.section`
width: 100%;
height: 100vh;
Copy link
Member

Choose a reason for hiding this comment

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

요고 height: auto 로 하는거 어떨까요? 푸터를 넣었을 때 auto로 해야 푸터가 맨 밑에 하단으로 가더라구요!!

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

수정했습니당

@intersoom intersoom merged commit 925cd79 into dev Aug 23, 2023
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.

2 participants