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

prompt dialog 구현 #167

Merged
merged 4 commits into from
Nov 28, 2024
Merged

prompt dialog 구현 #167

merged 4 commits into from
Nov 28, 2024

Conversation

hoqn
Copy link
Collaborator

@hoqn hoqn commented Nov 28, 2024

✏️ 한 줄 설명

이 PR의 주요 변경 사항이나 구현된 내용을 간략히 설명해 주세요.

window.prompt로 다뤄지던 기능을 자체적인 Dialog로 구현했어요

✅ 작업 내용

  • prompt 모듈을 구현해서, 자체적인 Dialog를 기존 prompt와 유사한 방법으로 사용할 수 있게 했어요.

🏷️ 관련 이슈

📸 스크린샷/영상

이번 PR에서 변경되거나 추가된 뷰가 있는 경우 이미지나 동작 영상을 첨부해 주세요.

2024-11-28.6.29.08.mov

📌 리뷰 진행 시 참고 사항

리뷰 코멘트 작성 시 특정 사실에 대해 짚는 것이 아니라 코드에 대한 의견을 제안할 경우, 강도를 함께 제시해주세요! (1점: 가볍게 참고해봐도 좋을듯 ↔ 5점: 꼭 바꾸는 게 좋을 것 같음!)

다음과 같이 사용할 수 있도록 해주었어요. async 함수라는 게 중요합니다.

const { name } = await prompt("제목 (null 가능)", "설명 (null 가능)", {
        label: "제목",
        name: "name",
        placeholder: "제목을 입력하세요",
      }, { /* ...여러 폼 같이 넣는 것도 됩니다... ...반환 값의 경우 타입 지원도 됩니다... */ });

@hoqn hoqn requested a review from a team as a code owner November 28, 2024 09:47
@hoqn hoqn requested review from heegenie and fru1tworld and removed request for a team November 28, 2024 09:47
@hoqn hoqn self-assigned this Nov 28, 2024
@heegenie heegenie added this to the Palette, Alert UI milestone Nov 28, 2024
Copy link
Collaborator

@heegenie heegenie left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 타입을 엄청 잘 활용해 주시는 것 같아요 ㅎㅎ
리뷰 읽으면서 저도 참고가 많이 되었습니다 😊

Copy link
Collaborator

Choose a reason for hiding this comment

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

오 Dialog 구현하신 김에 공유 기능도 추가해 주셨군요 ㅎㅎ 좋습니다 😄

@hoqn hoqn merged commit 068f443 into dev Nov 28, 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.

Alert UI를 구현한다.
3 participants