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

chore : 로그인 시 '로그인중..' 글자로 바꾸고 버튼 배경색 살짝 연해지도록 디자인 변경 #419

Merged

Conversation

godhyzzang
Copy link
Collaborator

@godhyzzang godhyzzang commented Dec 16, 2024

📄 Description of the PR

  • 기능 설명 : chore : 로그인 과정 사용자 경험 개선위해 로그인 시 '로그인중..' 글자로 바꾸고 버튼 배경색 살짝 연해지도록 디자인 변경

🔧 What has been changed?

📸 Screenshots / GIFs (if applicable)

로그인배경변경

⚠️ Precaution & Known issues

✅ Checklist

  • UI 브랜치 같이 확인해서 이슈없는지 확인해보기
  • 함수 이름, 변수 이름만 봐도 어떤 기능을 하는지 파악할 수 있는지 (선언적인 코드인지 확인)

@godhyzzang godhyzzang requested a review from smosco December 16, 2024 07:03
@godhyzzang godhyzzang self-assigned this Dec 16, 2024
@godhyzzang godhyzzang linked an issue Dec 16, 2024 that may be closed by this pull request
3 tasks
Copy link
Collaborator

@smosco smosco left a comment

Choose a reason for hiding this comment

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

혹시 수정해주실 수 있나요??

Comment on lines 40 to 46
<div className="relative min-h-screen bg-gradient-to-br from-indigo-50 to-violet-50 flex flex-col items-center justify-center p-4">
{isLoading && (
<div className="absolute inset-0 flex justify-center items-center z-50">
<LoadingSpinner />
</div>
)}
<div className={`max-w-md w-full space-y-8 ${isLoading && 'opacity-50'}`}>
Copy link
Collaborator

Choose a reason for hiding this comment

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

전체 로딩 스피너보다는 각 버튼 옆에 작게 흰색으로 로딩 스피너를 보여주는 게 좋을 것 같아요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

스피너 대신 버튼 자체의 텍스트를 변경하고 배경색을 연하게 만드는 방법으로 변경했습니다

Copy link
Collaborator

@smosco smosco left a comment

Choose a reason for hiding this comment

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

수정해주시면 approve 할게용

Comment on lines 83 to 85
{KakaoIsLoading && (
<LoadingSpinner size="small" className="absolute right-1/4" />
)}
Copy link
Collaborator

Choose a reason for hiding this comment

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

밀리면 그냥 로그인중으로 바꿔주세용

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

네엡 스피너 대신 버튼 자체의 텍스트를 변경하고 배경색을 연하게 만드는 방법으로 변경했습니다

@godhyzzang godhyzzang changed the title chore : 로그인 과정 사용자 경험 개선위해 로딩스피너 넣음 chore : 로그인 시 '로그인중..' 글자로 바꾸고 버튼 배경색 살짝 연해지도록 디자인 변경 Dec 16, 2024
@godhyzzang godhyzzang merged commit 6380043 into develop Dec 16, 2024
1 check passed
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.

[chore] add loading spinner when login procedure
2 participants