Skip to content

[Feat] create TodoModal (생성/수정)#33

Merged
gibeom0218 merged 32 commits into
mainfrom
27-feature-create-modal-component
Nov 28, 2024
Merged

[Feat] create TodoModal (생성/수정)#33
gibeom0218 merged 32 commits into
mainfrom
27-feature-create-modal-component

Conversation

@gibeom0218
Copy link
Copy Markdown
Member

@gibeom0218 gibeom0218 commented Nov 28, 2024

📄 TodoModal (생성/수정)

📝 변경 사항 요약

  • TodoContainer 추가 (이것으로 배경 지정하면 될듯)
  • 퍼블리싱 및 각 항목마다 컴포넌트 분리하였음
  • 모달 닫힘 기능과 버튼 체크 기능 구현
  • 생성 / 수정에 대한 임의의 props를 받아 모달의 종류를 구분

📌 관련 이슈

🔍 변경 사항 상세 설명

생성 / 수정에 대한 임의의 props를 받아 모달의 종류를 구분에 관한 부분은
언제든지 바뀔 수 있습니다. (url에서? zustand에서? props?)

모달 닫힘 기능은 배경과 x아이콘 클릭시 모두 발생하도록 하였음

✅ 확인 사항

  • 코드가 정상적으로 컴파일되는지 확인했습니다.
  • 관련 테스트를 작성하고 모두 통과했는지 확인했습니다.
  • 코드 스타일 가이드에 맞게 작성했습니다.

📸 스크린샷 (선택 사항)

할일 생성 수정 모달

기타 참고 사항

일단 스토리북은 작성 못하였습니다 ㅠㅠ... 추후에 추가하도록 하겠습니다.

@gibeom0218 gibeom0218 added the ✨ Feature 기능 개발 label Nov 28, 2024
@gibeom0218 gibeom0218 self-assigned this Nov 28, 2024
@gibeom0218 gibeom0218 linked an issue Nov 28, 2024 that may be closed by this pull request
@github-actions
Copy link
Copy Markdown

🎉 구현한 기능 Preview: https://front-kgi2rytip-beoms-projects-53e61468.vercel.app

Copy link
Copy Markdown
Contributor

@dudwns0213 dudwns0213 left a comment

Choose a reason for hiding this comment

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

쑤고하셨어용 LGTM

<span className="text-lg-bold text-slate-800">
{todoType === '생성' ? '할 일 생성' : '할 일 수정'}
</span>
<IoMdClose size={24} className="cursor-pointer" onClick={handleClose} />
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

size를 className 안에 넣을 수는 없나요?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

감사합니다!! 지금 해보니까 넣을 수 있는것 같더라구요!!
가독성을 위해 바꾸겠습니다

<div className="inline-flex flex-col items-center justify-center gap-8">
<FaLink size={24} className="text-slate-400" />
<span className="text-sm-normal text-slate-400 sm:text-base-normal">
링크를 첨부해주세요
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

이런 고정된 텍스트의 경우에는 constants 폴더 안에 따로 빼서 적용해도 괜찮지 않을까요?

<button className="inline-flex flex-col items-center justify-center gap-8">
<FaPlus size={24} className="text-slate-400" />
<span className="text-sm-normal text-slate-400 sm:text-base-normal">
파일을 업로드해주세요
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

여기도 따로 빼도 좋을거같아용

};
return (
<div
className="fixed inset-0 z-50 flex items-center justify-center bg-black/80"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

flex items-center justify-center를 자주 사용하실 거 같으면 globals에 따로 지정해서 간단하게 사용해도 괜찮을거같아요

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

넵 참고하겠습니다@@ 혹시 참고될만한 내용있으신가요?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

tailwind 공식문서 사용자 정의 함수

@layer components{
 .ex{
   @apply flex ...
   }
  }
 }

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

감사합니당

Comment thread src/app/(route)/GiBeom/page.tsx Outdated
<input
type="url"
placeholder="링크를 입력하세요"
className="mt-4 w-full max-w-[500px] rounded-8 border px-4 py-2 text-sm text-slate-600 focus:outline-none" // 길이 늘림
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.

max-w-[500px]에서 max-w-500은 적용이 안되나요??

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

엌ㅋㅋ 실수를 헀네요 ^^

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.

ㅋㅋㅋㅋㅋㅋㅋ굳

Comment thread src/components/common/Modal/TodoModal/TodoModalHeader/index.tsx Outdated
Comment thread src/components/common/Modal/TodoModal/index.tsx Outdated
@github-actions
Copy link
Copy Markdown

🎉 구현한 기능 Preview: https://front-404gtic65-beoms-projects-53e61468.vercel.app

@github-actions
Copy link
Copy Markdown

🎉 구현한 기능 Preview: https://front-567adgamy-beoms-projects-53e61468.vercel.app

@github-actions
Copy link
Copy Markdown

🎉 구현한 기능 Preview: https://front-ingl74m91-beoms-projects-53e61468.vercel.app

@gibeom0218 gibeom0218 force-pushed the 27-feature-create-modal-component branch from 3d100ea to 45a792e Compare November 28, 2024 06:42
@github-actions
Copy link
Copy Markdown

🎉 구현한 기능 Preview: https://front-91z6e88r4-beoms-projects-53e61468.vercel.app

@github-actions
Copy link
Copy Markdown

🎉 구현한 기능 Preview: https://front-82v6vqs1q-beoms-projects-53e61468.vercel.app

@gibeom0218 gibeom0218 force-pushed the 27-feature-create-modal-component branch from 45a792e to 832c18d Compare November 28, 2024 07:14
@github-actions
Copy link
Copy Markdown

🎉 구현한 기능 Preview: https://front-cipmyr850-beoms-projects-53e61468.vercel.app

@gibeom0218 gibeom0218 merged commit 57c9383 into main Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 기능 개발

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] modal 제작

3 participants