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

[FE] 비밀번호 입력 간소화 및 모바일 숫자 키패드를 적용 #342

Merged
merged 2 commits into from
Sep 5, 2024

Conversation

Yoonkyoungme
Copy link
Contributor

관련 이슈

작업 내용

1. 입력 필드 유효성 검사 규칙 개선 및 단순화

  • 각 필드(meetingName, nickname, password)의 유효성 검사 규칙을 정규표현식으로 단순화
  • 비밀번호 규칙을 4자리 숫자로 변경
  • 새로운 비밀번호 규칙(4자리 숫자)에 맞게 label 텍스트 수정

2. 모바일 숫자 키패드 적용

  • input type="number" 적용
  • iOS에서는 추가 속성 적용 (inputMode="numeric"pattern="[0-9]*" 속성을 추가하여 iOS 기기에서 숫자 전용 키패드가 표시되도록 개선)

참고 자료

작업 현황 정리 노션 링크

특이 사항

비밀번호 입력이 4자리 숫자로 변경되고 모바일 키패드 입력이 도입됨에 따라 PasswordInput 컴포넌트(비밀번호 숨기기/표시 기능이 포함된 컴포넌트)는 현재 사용되지 않고 있습니다.
그러나 추후에 필요할 수 있으므로 컴포넌트는 삭제하지 않고 남겨두었습니다.

리뷰 요구사항 (선택)

@Yoonkyoungme Yoonkyoungme added 🐈 프론트엔드 프론트엔드 관련 이슈에요 :) 🚀 기능 기능을 개발해요 :) ♻️ 리팩터링 코드를 깎아요 :) labels Sep 4, 2024
@Yoonkyoungme Yoonkyoungme added this to the 5차 데모데이 milestone Sep 4, 2024
@Yoonkyoungme Yoonkyoungme self-assigned this Sep 4, 2024
Copy link

github-actions bot commented Sep 4, 2024

Test Results

5 tests   5 ✅  3s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit 4df0b5e.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@hwinkr hwinkr left a comment

Choose a reason for hiding this comment

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

🔮 빙봉뱅봉!

레벨 4 첫 작업 고생하셨어요~ 👏👏👏

Comment on lines +83 to +84
inputMode="numeric"
pattern="[0-9]*"
Copy link
Contributor

Choose a reason for hiding this comment

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

[C]

오, 이렇게 하면 ios 디바이스에서도 숫자 키패드를 띄울 수 있군요! 하나 배워갑니다

Copy link
Contributor

@Largopie Largopie left a comment

Choose a reason for hiding this comment

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

대 빙 봉 행차하셨습니다 👍

TERN으로 변경하여 정규표현식 사용
- 각 필드의 유효성 검사 규칙을 정규표현식으로 단순화
- 비밀번호 규칙을 4자리 숫자로 변경
- 비밀번호 input type을 number로 설정
@Yoonkyoungme Yoonkyoungme force-pushed the feat/339-simplify-password-and-mobile-keypad branch from 5802afa to 4df0b5e Compare September 5, 2024 05:53
@Yoonkyoungme Yoonkyoungme merged commit 97438f7 into develop Sep 5, 2024
5 checks passed
@Yoonkyoungme Yoonkyoungme deleted the feat/339-simplify-password-and-mobile-keypad branch September 5, 2024 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ 리팩터링 코드를 깎아요 :) 🐈 프론트엔드 프론트엔드 관련 이슈에요 :) 🚀 기능 기능을 개발해요 :)
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FE] 비밀번호 입력 간소화 및 모바일 숫자 키패드를 적용해요 :)
3 participants