Skip to content

Conversation

@handje
Copy link
Collaborator

@handje handje commented Jan 30, 2025

Pull request

Related issue

Resolve #61 @handje

Motivation and context

  • 주식리스트 테이블을 보여주는 메인페이지 레이아웃을 작업하였습니다.
  • 임시 헤더를 실제 헤더로 적용하였습니다.

Solution

헤더

  1. shadcn컴포넌트에 의해 클라이언트 컴포넌트를 사용해야하는 Header컴포넌트와 header태그를 분리하였습니다.
  2. 자연스러운 fallback효과를 위해 내용을 수정하였습니다. (로고->주톡피아, avatar->user아이콘)

주식리스트

  1. 주식 리스트와 상세주식페이지를 분리하기 위해 파일명을 수정하였습니다. 이부분은 더 좋은 네이밍이 있다면 의견 주시면 감사하겠습니다 :)
  2. 현재 메인페이지에는 테이블만 있어 디자인적으로 좋지 못하여, 추후에 디자인을 수정할 예정입니다.
  3. 주식 종목이 고정되었기에 meta데이터는 정적으로 선언하였습니다.
  4. @KimKyuHoi 디자인시스템에는 DataTable이 아닌 Table만 남기고 DataTable의 내용을 실제 컴포넌트로 가져올 예정입니다. 이 부분에서 ui를 넘어가는 부분이 될 듯 하여 이슈를 분리하여 주식리스트 테이블 레이아웃 구현 #90 진행할 예정입니다.

How has this been tested

  • 상세페이지 구현 편의상 버튼은 그대로 남겨두었습니다. 테이블 요소 링크 연결 후 삭제하겠습니다.
    image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the docs/CONTRIBUTING.md document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@handje handje requested a review from KimKyuHoi January 30, 2025 15:02
@handje handje self-assigned this Jan 30, 2025
@handje handje added ✨ Feature 기능 추가 💄 Design 프론트엔드 CSS 관련 이슈 🖥️ FE 프론트엔드 ♻️ Need To Refactor 추후 리팩토링이 필요한 이슈 및 논의 무조건 스프린트내에 해야하는 것들 labels Jan 30, 2025
@handje handje added this to the 주톡피아 마일스톤 1 milestone Jan 30, 2025
Copy link
Collaborator

@KimKyuHoi KimKyuHoi left a comment

Choose a reason for hiding this comment

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

요거 몇개만 수정해주시면 좋을것 같네요!

@KimKyuHoi
Copy link
Collaborator

KimKyuHoi commented Jan 30, 2025

뭔가 그리고 드는 생각이지만 랜딩페이지에 테이블 바로 위에 로고랑 저희 깃헙 링크 올라가면 조금 덜 휑해 보이지 않을까,,,라는 생각이긴합니다 하핫 근데 이거는 리팩 때 생각해볼까요?

@handje
Copy link
Collaborator Author

handje commented Jan 31, 2025

#90 pr내용에 써둔 해당 이슈 확인부탁드립니다!
말씀해주신 내용 모두 수정과 유연성을 높이기 위해 디자인시스템의 DataTable을 없애고 Table만을 사용할 예정입니다.
그 부분에서 link를 연결할 예정이라 다음 pr에서 수정하겠습니다!
그 전까지는 규회님이 상세 페이지를 구현하실동안 사용하셔야하기때문에 임시 버튼을 남겨두었습니다!

@handje
Copy link
Collaborator Author

handje commented Jan 31, 2025

뭔가 그리고 드는 생각이지만 랜딩페이지에 테이블 바로 위에 로고랑 저희 깃헙 링크 올라가면 조금 덜 휑해 보이지 않을까,,,라는 생각이긴합니다 하핫 근데 이거는 리팩 때 생각해볼까요?

넵 좋아요! 로고도 없고 그래서 더 휑하긴 하네요 😮‍💨

@handje handje requested a review from KimKyuHoi January 31, 2025 00:49
Copy link
Collaborator

@KimKyuHoi KimKyuHoi left a comment

Choose a reason for hiding this comment

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

#90 pr내용에 써둔 해당 이슈 확인부탁드립니다! 말씀해주신 내용 모두 수정과 유연성을 높이기 위해 디자인시스템의 DataTable을 없애고 Table만을 사용할 예정입니다. 그 부분에서 link를 연결할 예정이라 다음 pr에서 수정하겠습니다! 그 전까지는 규회님이 상세 페이지를 구현하실동안 사용하셔야하기때문에 임시 버튼을 남겨두었습니다!

아 넵 좋은것 같습니다. 그렇게 하시죠!

@handje handje merged commit c52d6bf into dev Jan 31, 2025
2 of 3 checks passed
@handje handje deleted the fe-feat/mainpage-layout branch February 2, 2025 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💄 Design 프론트엔드 CSS 관련 이슈 🖥️ FE 프론트엔드 ✨ Feature 기능 추가 ♻️ Need To Refactor 추후 리팩토링이 필요한 이슈 및 논의 무조건 스프린트내에 해야하는 것들

Projects

None yet

Development

Successfully merging this pull request may close these issues.

메인화면(주식테이블) 레이아웃 구현

3 participants