-
Notifications
You must be signed in to change notification settings - Fork 0
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
Conversation
There was a problem hiding this 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} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아이콘이 얇아 잘 보이지 않는 것 같아 다른 아이콘 또는 여기를 참고해 Tabler를 사용하면 좋을 것 같습니다!
There was a problem hiding this comment.
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} />} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pin 아이콘도 Tabler에 같은 모양이 있으니 여기를 참고해 편하게 쓰실 수 있을 것 같습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정했습니다!
늦은 수정 죄송합니다. 마지막 페이지의 경우 dummyRow가 추가로 보여지도록 해서 페이지네이션을 고정시켰습니다. 수정된 코드 확인 부탁드립니다! |
There was a problem hiding this 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; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
부모 스타일에 맞게 유동적으로 너비가 변할 수 있도록 width를 고정하지는 않아도 괜찮을 것 같습니다!
작성자: @sera2002
이슈: #8
체크 리스트
작업 내역
비고
close/resolve/fix #{이슈 번호 기입}