Skip to content

Conversation

@0-hu
Copy link
Contributor

@0-hu 0-hu commented May 27, 2025

📌 관련 이슈번호

✏️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

✅ Check List

  • 🔀 PR 제목의 형식을 잘 작성했나요?
  • 🏗️ yarn build는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요? (TODO, 주석, clg... etc.)
  • ✅ 컨벤션을 지켰나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 🙇‍♂️ 리뷰어를 지정했나요?

✨ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. 작업 내용

📢 To Reviewers

  • 카드 컴포넌트 rounded 수정
    image

  • small 사이즈 카드 최대 표시 태그 수정(조언받기 페이지에서 카드 넘치는 문제)
    image

호버에 태그 모두 표시
image

  • TagSearchBar 콘솔 로그 제거
    콘솔 로그 찍히는거 제거했습니다.

📸 스크린샷

위 첨부 스크린샷 참조

🔗 참고 자료

Copy link
Contributor

@sinji2102 sinji2102 left a comment

Choose a reason for hiding this comment

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

image

지금 카드 컴포넌트의 클릭 상태를 로컬 스토리지에 저장하고 있는데, 만약 모달을 열고 다른 페이지로 이동하거나 모달을 연 상태로 새로고침하면 모달을 닫았다는 판단이 전달되지 않고 있어 클릭이 유지되고 있어. 단순 상태를 로컬 스토리지에 저장하는 것도 좋은 선택이 아니기도 하고, 상태가 유지되는 오류가 있어서 state로 저장해야 할 것 같아요.

@0-hu
Copy link
Contributor Author

0-hu commented May 28, 2025

state 기반으로 동작하도록 수정했습니다.

@0-hu 0-hu requested a review from sinji2102 May 28, 2025 08:49
Copy link
Contributor

@sinji2102 sinji2102 left a comment

Choose a reason for hiding this comment

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

확인했습니다 머지해도 될 것 같아요!

@0-hu 0-hu merged commit 95c72a0 into develop May 28, 2025
@sinji2102 sinji2102 deleted the refactor/#78/FixCardComponent branch May 28, 2025 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ Refactor ] 카드 컴포넌트 수정

3 participants