[FE] feat: 소셜 연동 로그인 버튼 및 로그인 요청 모달 #1022
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🚀 어떤 기능을 구현했나요 ?
🔥 어떻게 해결했나요 ?
LoginButton의 props
나중에 깃허브 외의 다른 소셜이 추가될 수 있다고 생각해서 공통 컴포넌트로 만들었습니다.
platform
과engPlatform
: 소셜 이름 뒤에 조사를 붙이기 위해 2개로 나눴습니다. 조사를 구해주는 유틸 함수가 한글 기준이라 영문이 오면 조사를 제대로 표현할 수 없었습니다. 그래서 유저에게 표시할 영문(optional)과 조사 유틸 호출을 위한 한글 표현을 모두 받습니다.$logoStyle
의 정체: 로고의 크기를 조절하고 싶을 때 사용합니다.LoginButton
이 공통 Button 컴포넌트를 사용하는데, 버튼의 스타일은 Button의$style
로 줄 수 있습니다.LoginButton
단에서 스타일 속성을 따로 만들어줬습니다.LoginRequestModal 구현 방법 관련
상황에 따라 모달 타이틀이 다릅니다. ('로그인하시겠어요?' 와 '회원이신가요?')
그래서 타이틀을 children으로 받을까 했지만, '로그인 요청 모달'이라는 로그인 특화 모달이라서 타이틀 문구를 상수나 타입으로 모달 안에서 관리하는 게 더 낫다고 생각했습니다.
지금은 간단하게 모달 안에서 type으로 사용하고 있습니다.
📝 어떤 부분에 집중해서 리뷰해야 할까요?
ContentModal
을 사용해서 만들었습니다.ㄴ 참고) 사진은 왠지 오른쪽이 좁아 보이는데 실제로 보면 중앙정렬은 맞습니다.
📚 참고 자료, 할 말
메리크리~