[#53] rolling paper list page 데스크탑 버전 완성 (아마도), 이미지 로딩 느린 문제, 나도 만들어보기 등 url 연결 제외 #54
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
(pr수정사항 누락된부분 추가 수정)
rolling paper list 페이지, 내부 컴포넌트 분리 및 데이터 정렬
… 및 reaction 표시
…a 연동 API 연동은 별도로 진행할 예정입니다.
…s container 너비 설정 `+n`에서 `n`이 10 이상이면 avatar와 달리 너비가 28px보다 커질 수 있으므로 실제 너비를 계산합니다.
(pr수정사항 누락된부분 추가 수정)
rolling paper list 페이지, 내부 컴포넌트 분리 및 데이터 정렬
삼한 연산자 변경, transient prop사용, styled-component를 최대한 사용하도록 변경, strong태그 변경
이모지등ㄷ 추가사항 적용
1 task
…-FE-18-part2#43 [codeit-FE-18-part2#43] rolling paper list 페이지, 내부 컴포넌트 분리 및 데이터 정렬
cskime
reviewed
Aug 17, 2025
Comment on lines
+167
to
+188
| const CardEmoji = styled.span` | ||
| background-color: rgba(0, 0, 0, 0.54); | ||
| border-radius: 32px; | ||
| color: #ffffff; | ||
| padding: 8px 12px; | ||
| margin-right: 8px; | ||
|
|
||
| position: relative; | ||
| white-space: nowrap; | ||
| overflow: hidden; | ||
|
|
||
| min-width: ${(props) => (props.$isLong ? "60px" : "auto")}; | ||
| max-width: ${(props) => (props.$isLong ? "44px" : "none")}; | ||
|
|
||
| transition: max-width 0.3s ease, z-index 0s; | ||
|
|
||
| &:hover { | ||
| max-width: 200px; | ||
| z-index: 100; | ||
| position: relative; | ||
| } | ||
| `; |
There was a problem hiding this comment.
직접 만드는 대신 EmojiBadge 공통 컴포넌트를 사용할 수 없는걸까요?
There was a problem hiding this comment.
직접 만드는 대신
EmojiBadge공통 컴포넌트를 사용할 수 없는걸까요?
근데 이게 공통컴포넌트를 사용하게되면 hover스타일은 적용할수없게되는데 그냥 없애라는 말씀이신가요?
이렇렇게 되거든요. 뭐 gap이야 추가할수있겠지만
…하는 workflow 추가 Vercel의 조직 repository 유료 plan을 우회하기 위해 forked repo를 통해 배포되도록 설정
gummmmmy0v0
approved these changes
Aug 19, 2025
이모지등ㄷ 추가사항 적용
cskime
approved these changes
Aug 19, 2025
Emoji 공통컴포넌트는 일단 주석처리
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 작업 내용
📷 스크린샷 (선택)
💬 리뷰어에게 남길 말 (선택)
이모지 count 예시데이터에 그렇게 큰 숫자가 들어갈거라고 생각하지는 않지만..
글자수가 3글자 이상이되면 카드 정렬이 깨지는 문제가 있어서 마우스 hover시에 숫자를 보이도록 했습니다.
혹시 구리다면 말씀해주세요. 아래줄로 넘기거나 gab을 줄인다던가 해봤는데 다 너무 별로더라구요. 🤔