Conversation
| <div className="mr-2 flex h-[120px] w-[85px] bg-middleBlack"> | ||
| <Logo className="m-2 w-full" /> | ||
| </div> | ||
| <div className="relative grow p-2"> |
There was a problem hiding this comment.
오호! grow 사용해본 적이 없어서 몰랐는데 이렇게 사용할 수 있군요! 👍
There was a problem hiding this comment.
헤헷! 저번에 상자가 옆에 상자 잡아먹는다고 질문드린건데, flex-grow하니까 해결되더라구요! 홋홋
src/pages/Library/Card/BookCard.tsx
Outdated
| <Typography variant="paragraph" component="div" className="!mb-2 font-semibold"> | ||
| Beginning Linux programming 4th Edition | ||
| </Typography> |
There was a problem hiding this comment.
단순히 텍스트만 들어가는 부분인 것 같은데 component div로 지정해준 이유가 있나요?~?
There was a problem hiding this comment.
그게 Typography variant="paragraph"이 inline으로 들어가더라구요!
근데 필요한건 block이라 div로 지정해줬습니다!
아예 variant="paragraph"를 빼버리면 block으로 들어가는데, 개인적으로 paragraph인걸 명시하는게 좋아보여서 이렇게 했습니다!
There was a problem hiding this comment.
움움 근데 variant="paragraph"를 명시적으로 안붙혀주는게 깔끔할 것 같기도 하구,, 의견 부탁드립니다!
There was a problem hiding this comment.
아하! 저는 paragraph는 기본값이니 따로 안 쓰긴하는데 적어줘도 좋은 것 같아요!
block요소 여부 때문인 거면 태그를 별도로 지정해주는 방식 말고 className에 block 추가해주는 건 어떤가요?
+) component로 태그 변경하면 개발자도구 element에서도 변경되는 지 이때까지 신경을 안쓰고 썼던 것 같은데 다음에 쓸때 한 번 확인해봐야겠네욤! 궁금쓰~ 😄
src/pages/Library/Library.tsx
Outdated
| <PageTitle>도서검색</PageTitle> | ||
| <div className="mb-5 flex w-full items-center justify-between"> | ||
| <SearchSection /> | ||
| <div className="mr-6 text-pointBlue">신청 가능 권수 : 1/5</div> |
There was a problem hiding this comment.
이부분도 단순 text 띄워주는 거면 시멘틱하게 Typography 사용해주면 어떨까요?
There was a problem hiding this comment.
넹! 나중에 text hover시 툴팁 띄워주게 할거긴 합니당!
…nt-R2 into feature/도서검색_ui_구현_#369 # Conflicts: # src/router/useMainRouter.tsx
연관 이슈
작업 요약
Preview 이미지