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

[FEAT] 관리자 게시판 미리보기 컴포넌트 추가 #40

Merged
merged 5 commits into from
Aug 11, 2024

Conversation

sera2222
Copy link
Contributor

작성자: @sera2002
이슈: #8

체크 리스트

  • 적절한 제목으로 수정했나요?
  • 상단에 이슈 번호를 기입했나요?
  • Target Branch를 올바르게 설정했나요?
  • Label을 알맞게 설정했나요?

작업 내역

  • 관리자 게시판 미리보기 컴포넌트를 개발하였습니다.

비고

  • 페이지 넘길 때 페이지네이션이 올라갔다 내려갔다 하는게 별로 좋아보이지 않는데 페이지네이션 위치를 고정하는 게 좋을까요?
  • 컴포넌트 전체 높이도 아예 높이를 고정시키는게 좋을지 모르겠습니다.

close/resolve/fix #{이슈 번호 기입}

@sera2222 sera2222 added the feat label Jul 26, 2024
@sera2222 sera2222 self-assigned this Jul 26, 2024
Copy link

@chunzhi23 chunzhi23 requested review from chunzhi23 and obinox July 27, 2024 11:33
Copy link
Contributor

@chunzhi23 chunzhi23 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.

페이지를 넘길 때 페이지네이션의 위치가 바뀌는 게 개인적으론 불편하다고 느껴 어느 정도 고정시켜두는 것이 편할 것 같다고 생각합니다.

페이지에 이 컴포넌트를 적용할 때 주어진 공간의 100%를 기본적으로 이 컴포넌트가 차지하도록 해서 가장 밑단에 페이지네이션을 고정시키고, 리스트가 100%를 넘기게 되면 그 때 리스트의 height에 따라 움직이게 하면 어떨까요?

<Stack gap={0} align="flex-start" justify="flex-start" className={classes.container}>
<Flex className={classes.title_bar}>
<div className={classes.col_icon}>
<Image alt="table icon" src="/icons/Table.svg" width={24} height={24} />
Copy link
Contributor

Choose a reason for hiding this comment

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

아이콘이 얇아 잘 보이지 않는 것 같아 다른 아이콘 또는 여기를 참고해 Tabler를 사용하면 좋을 것 같습니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정했습니다!

onClick={onClickRow}
>
<div className={classes.column_pinned}>
{pinned && <Image src="/icons/Pin_fill.svg" alt="pinned" width={24} height={24} />}
Copy link
Contributor

Choose a reason for hiding this comment

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

Pin 아이콘도 Tabler에 같은 모양이 있으니 여기를 참고해 편하게 쓰실 수 있을 것 같습니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정했습니다!

@sera2222
Copy link
Contributor Author

sera2222 commented Aug 9, 2024

고생하셨습니다.

페이지를 넘길 때 페이지네이션의 위치가 바뀌는 게 개인적으론 불편하다고 느껴 어느 정도 고정시켜두는 것이 편할 것 같다고 생각합니다.

페이지에 이 컴포넌트를 적용할 때 주어진 공간의 100%를 기본적으로 이 컴포넌트가 차지하도록 해서 가장 밑단에 페이지네이션을 고정시키고, 리스트가 100%를 넘기게 되면 그 때 리스트의 height에 따라 움직이게 하면 어떨까요?

늦은 수정 죄송합니다. 마지막 페이지의 경우 dummyRow가 추가로 보여지도록 해서 페이지네이션을 고정시켰습니다. 수정된 코드 확인 부탁드립니다!

Copy link
Contributor

@chunzhi23 chunzhi23 left a comment

Choose a reason for hiding this comment

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

확인 감사합니다!
아래 한 가지 요청사항만 확인해주세요 :)

@@ -0,0 +1,42 @@
.container {
width: 600px;
Copy link
Contributor

Choose a reason for hiding this comment

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

부모 스타일에 맞게 유동적으로 너비가 변할 수 있도록 width를 고정하지는 않아도 괜찮을 것 같습니다!

@sera2222 sera2222 merged commit c775da8 into develop Aug 11, 2024
1 check passed
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants