Skip to content
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

[feature] 이벤트 및 그룹 카드 선택 시 border 유지 #67

Merged
merged 6 commits into from
Sep 22, 2024

Conversation

seongwon030
Copy link
Collaborator

@seongwon030 seongwon030 commented Sep 22, 2024

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가 ✨
  • 기능 삭제 🔥
  • 버그 수정 🐛
  • 코드 형태 개선 🎨
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트 🔨

변경 사항

🌱이벤트카드 및 그룹 카드 className설정

  • ✔️className으로 클릭한 카드를 전달하기 위해
//이벤트카드
   <Styled.Card
          key={event.eventId}
          onClick={() => handleCardClick(event.eventId)}
          className={selectedEventId === event.eventId ? 'selected' : ''}
        >

//그룹카드
<Styled.GroupCard
          onClick={() => handleCardClick(group.bandId)}
          className={selectedBandId === group.bandId ? 'selected' : ''}
        >

스타일 컴포넌트

 &.selected {
    border: 3px solid #4e54f5;
  }

🌱클릭이벤트 변경

  • ✔️ 카드 클릭시 -> 이미지 클릭시 수정폼 보이도록 변경
//이벤트 카드
 <img
              onClick={() => handleCardClick(group.bandId)}
              src="/images/EditEvent.png"
              alt="EditEvenIcon"
            />
//그룹카드
 <img
              onClick={() => handleCardClick(group.bandId)}
              src="/images/EditEvent.png"
              alt="EditEvenIcon"
            />

이슈 사항

To reviewer

[PN]을 활용해서 리뷰를 남겨주세요!
P1: 꼭 반영해 주세요 (Request changes)
P2: 웬만하면 반영해 주세요 (Comment)
P3: 그냥 사소한 의견입니다 (Approve)

ex) [P3]여기에서 이 부분 잘 모르겠는데 한번 봐주실 수 있나요?

@seongwon030 seongwon030 changed the title [feature] 이벤트 및 그룹 카드 선택 시 border 유지 [feature] 이벤트 및 그룹 카드 선택 시 border 유지 및 카드클릭 이벤트 수정 Sep 22, 2024
@seongwon030 seongwon030 changed the title [feature] 이벤트 및 그룹 카드 선택 시 border 유지 및 카드클릭 이벤트 수정 [feature] 이벤트 및 그룹 카드 선택 시 border 유지 Sep 22, 2024
Copy link
Collaborator

@wkdghdwns199 wkdghdwns199 left a comment

Choose a reason for hiding this comment

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

[P3] 아주 좋은 기능 추가인것 같습니다 수고하셨습니다 😎

Copy link
Collaborator

@wkdghdwns199 wkdghdwns199 left a comment

Choose a reason for hiding this comment

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

[P3] : 수고하셨습니다 👍

Copy link
Member

@oesnuj oesnuj left a comment

Choose a reason for hiding this comment

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

[P3] 카드 클릭에서 아이콘 클릭으로!!!
엄청납니다💪

@seongwon030 seongwon030 merged commit ee5d52f into develop Sep 22, 2024
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.

[feature] 이벤트 카드 클릭시 테두리 유지
3 participants