Skip to content

Comments

✨ feat: 댓글 & 대댓글 컴포넌트 구현#51

Open
10hajin15 wants to merge 1 commit intomainfrom
40-feat-comment-component
Open

✨ feat: 댓글 & 대댓글 컴포넌트 구현#51
10hajin15 wants to merge 1 commit intomainfrom
40-feat-comment-component

Conversation

@10hajin15
Copy link
Contributor

📍 PR 타입 (하나 이상 선택)

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타 사소한 수정

❗️ 관련 이슈 [#]

📄 개요

  • 댓글 컴포넌트를 구현하였습니다.

🔁 변경 사항

  • 댓글(&대댓글) 컴포넌트 구현
  • RightBottomIcon 추가

📸 스크린샷

[기본 댓글]
스크린샷 2025-01-09 오후 2 38 11

[대댓글] - 2번째 스크린샷은 크기를 보기 편하게 캡처할 때만 bg를 임의적으로 바꾸었습니다.
스크린샷 2025-01-09 오후 2 38 17
스크린샷 2025-01-09 오후 2 38 34

👀 기타 더 이야기해볼 점

@10hajin15 10hajin15 added the ✨ feat 기능 개발 label Jan 9, 2025
@10hajin15 10hajin15 self-assigned this Jan 9, 2025
@10hajin15 10hajin15 linked an issue Jan 9, 2025 that may be closed by this pull request
2 tasks
@GuKBABjOa
Copy link
Contributor

하진님, 수고하셨습니다

Copy link
Contributor

@junhakjh junhakjh left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 :)

export { LeftArrowIcon, RightArrowIcon, UpArrowIcon, DownArrowIcon };
const RightBottomArrowIcon: React.FC<PArrowIcon> = () => {
return (
<svg width='23' height='24' viewBox='0 0 23 24' fill='none' xmlns='http://www.w3.org/2000/svg'>
Copy link
Contributor

Choose a reason for hiding this comment

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

widthheight의 비율이 1:1이 아닌 것은 의도된 부분인가요?

import { TTier } from '@/model/tier';

interface PComment {
data: {
Copy link
Contributor

Choose a reason for hiding this comment

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

따로 타입을 만들면 좋을 것 같아요!
ex) TMiniProfile

Copy link
Contributor

Choose a reason for hiding this comment

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

그리고 data라는 속성명보다 좀 더 직관적인 속성명을 사용하면 더욱 직관적으로 코드를 이해할 수 있을 것 같아요 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feat 기능 개발

Projects

None yet

Development

Successfully merging this pull request may close these issues.

✨ feat: Comment Component

3 participants