Skip to content

[REFACTOR] 모달 리팩토링#78

Merged
gaeulzzang merged 2 commits into
developfrom
feature/#67-modal-refactor
Jul 26, 2025
Merged

[REFACTOR] 모달 리팩토링#78
gaeulzzang merged 2 commits into
developfrom
feature/#67-modal-refactor

Conversation

@gaeulzzang

Copy link
Copy Markdown
Collaborator

체크리스트

  • 🧰 npm run dev로 실행 환경에서 잘 돌아가는걸 확인했나요?
  • 🎋 base 브랜치를 develop 브랜치로 설정했나요?
  • 🖌️ PR 제목은 형식에 맞게 잘 작성했나요?
  • 🏷️ 라벨은 등록했나요?
  • 🙆 리뷰어는 등록했나요?

📌 관련 이슈번호


✅ Key Changes

  • 모달 props 추가 (버튼 텍스트, 버튼 색상, 버튼 클릭 메소드)
  • red-400 색상 error-400으로 이름 변경

📸 스크린샷 or 실행영상

image

💬 To Reviewers

늦게 올리게 되어 죄송합니다. 다시 열심히 개발 하겠습니다 🥲
아래 코드는 모달 사용법입니다. 모달 사용하시는 분들은 피그마에 맞게 변경 부탁드립니다.
부제목에 줄바꿈이 있을 경우 아래와 같이 <></> 태그로 감싸주시면 되고 줄바꿈이 없을 경우 string값 전달해주시면 됩니다.

return (
    <Modal
      title="알림"
      subtitle={
        <>
          정말로 삭제하시겠습니까?
          <br />
          삭제된 데이터는 복구할  없습니다.
        </>
      }
      buttonText="삭제"
      buttonColor="bg-red-400"
      onClick={() => console.log('Modal Button Clicked')}
    />
  );

@gaeulzzang gaeulzzang self-assigned this Jul 26, 2025
@gaeulzzang gaeulzzang added ♻️ REFACTOR 코드 수정 및 리팩토링 🍁 가을 labels Jul 26, 2025
@github-actions

Copy link
Copy Markdown

@waldls waldls left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

확인했습니다 :) 온보딩에서도 #d44242 컬러가 필요했는데, 코드로 따로 정의해주셔서 감사합니다! 유용하게 쓰겠습니다. 고생 많으셨어요!

@sunhwaaRj sunhwaaRj left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

제 파트에서 사용되는 해당 색상이랑 모달창 추후 변경하여 적용하도록 하겠습니다 수고하셨습니다~~

@gaeulzzang
gaeulzzang merged commit 3329a52 into develop Jul 26, 2025
1 check passed
@gaeulzzang
gaeulzzang deleted the feature/#67-modal-refactor branch July 27, 2025 09:55
@HiJuwon HiJuwon added this to the Demo Day milestone Aug 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

♻️ REFACTOR 코드 수정 및 리팩토링 🍁 가을

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 공통 모달 수정

4 participants