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

[Feat/#32] 팝업 시트 컴포넌트 제작 #33

Merged
merged 6 commits into from
Feb 2, 2025
Merged

[Feat/#32] 팝업 시트 컴포넌트 제작 #33

merged 6 commits into from
Feb 2, 2025

Conversation

sikkzz
Copy link
Collaborator

@sikkzz sikkzz commented Feb 1, 2025

💡 변경사항 & 이슈

바텀 팝업 시트 컴포넌트 제작
해시태그 선택 페이지 시트 연결

✍️ 관련 설명

팝업 시트는 우선 radix-ui 사용했어. 직접 구현할 수도 있겠지만 이건 짧게 걸릴 시간은 아닐 수도 있어서 라이브러리 사용했고
이후에는 라이브러리 없이 직접적으로 구현할 거 같기는해!

이외에는 요구사항에 맞춰서 구현했어

  • 해시태그 선택 페이지에서 +아이콘 클릭 시 팝업 시트 오픈
  • 팝업 시트 오픈 시 input autoFocus
  • input은 20글자까지, 20글자 넘을 시 버튼 disabled, input 에러처리

⭐️ Review point

  • 요구사항 놓친 부분 있는지
  • 로직이나 코드 개선점

📷 Demo

스크린샷 2025-02-01 오후 10 05 05

@sikkzz sikkzz added the Feature new feature label Feb 1, 2025
@sikkzz sikkzz self-assigned this Feb 1, 2025
@sikkzz sikkzz requested a review from lgrin-byte as a code owner February 1, 2025 13:05
@sikkzz sikkzz changed the base branch from feat/#25 to develop February 1, 2025 13:06
Copy link

github-actions bot commented Feb 1, 2025

🧷 Storybook: https://6798a432c75b75bf6b81bc26-bnyxdsmrxq.chromatic.com/

⏰ Update: 2025년 02월 02일 23시 52분

@sikkzz sikkzz linked an issue Feb 1, 2025 that may be closed by this pull request
2 tasks
Copy link
Member

@lgrin-byte lgrin-byte left a comment

Choose a reason for hiding this comment

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

화면 넓을때는 뒤에 ui가 깨지는 것 같은데 크리티컬한 건 아닌듯!

@sikkzz
Copy link
Collaborator Author

sikkzz commented Feb 2, 2025

화면 넓을때는 뒤에 ui가 깨지는 것 같은데 크리티컬한 건 아닌듯!

우선 해결했어! radix-ui에 제공하는 dialog global style하고 충돌이 나서 그런거더라구. 이후에 바꾸고 싶은 욕구가 더 들게 만드는 거 같아 일단 임시로 global.scss에 처리해놨어

@sikkzz sikkzz merged commit 7b9b852 into develop Feb 2, 2025
3 checks passed
@sikkzz sikkzz deleted the feat/#32 branch February 2, 2025 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 팝업 시트 컴포넌트 제작
2 participants