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

✨ 로그인 버튼 추가 및 폰트 적용 #4

Merged
merged 2 commits into from
Feb 9, 2024

Conversation

changhwan77
Copy link
Collaborator

@changhwan77 changhwan77 commented Feb 7, 2024

Title

✨ 로그인 버튼 추가 및 폰트 적용

What type of PR is this?

  • 🍕 Feature ( 새로운 기능 추가 )
  • 🐛 Bug Fix ( 버그 수정 )
  • 📝 Documentation Update ( 개발자 관련 문서 업데이트 )
  • 🎨 Style ( 코드 포맷팅, 세미콜론 누락, 코드 변경이 없는 경우 )
  • 🧑‍💻 Code Refactor ( 코드 리펙토링 )
  • 🔥 Performance Improvements ( 성능 향상 )
  • ✅ Test ( 테스트 코드, 리펙토링 테스트 코드 추가 )
  • 🤖 Build ( 빌드 )
  • 🔁 CI
  • 📦 Chore (Release) (빌드 업무 수정, 패키지 매니저 수정 )
  • ⏩ Revert ( 커밋을 날린 경우 )

Description

  1. 로그인 버튼 추가 및 클릭 시 HomeScreen으로 이동.
  2. 폰트 변경에 따른 폰트 적용
  • SB 어그로 폰트 변경 -> Light, Medium, Large 3가지. -> 각 weight 3, 4, 700 적용했습니다.
  • 위 폰트 크기에 맞춰 Tuti Text Small, Medium, Large 에 300 ,400, 700 적용했습니다.

Related Issue number and link

  • Jira :
  • Issue :

Mobile & Desktop Screenshots/Recordings

default.mp4

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • Any dependent changes have been merged and published in downstream modules

To Reviewers

  • 추후 로그인 안내 메시지 추가 예정입니다!

@changhwan77 changhwan77 self-assigned this Feb 7, 2024
}
return name;
}

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

  1. getDisplayName
    비로그인 유저의 경우(토큰이 있는지 체크) 이름의 중간을 가리도록 로직을 걸어줬어요.

  2. _maskName

  • 비로그인 유저의 이름(name)을 전달 받아 2번째 Character를 기호(*)로 바꿔주는 함수.

snapshot.data ?? '',
);
}
}),
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

_getDisplayName 이 유저가 토큰이 있는지 없는지 체크하는 행위가 비동기 작업므로
FutureBuilder를 사용해 future 값에 할당하고 작업이 완료되면 해당 이름을 TuTi.Text에 할당했어요.

color: Colors.white,
fontWeight: FontWeight.w100,
),
),
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

로그인 버튼 > 클릭 시 HomeScreen으로 라우팅 하게 만들었어요.

다만, 추후 로그인 안내 메시지 구현 시 > 안내 메시지를 띄우도록 로직을 변경할 예정이에요.

// }
// }
// return null;
// },
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

<비로그인 유저도 트티 화면에 접속하게 만들기 위한 주석 처리>

  • 위 로직은 비로그인 유저가 위에서 표시한 화면 이외에 접속하고 있을 경우 HomeScreen으로 redirect 하는 로직이므로 일단은 주석처리로 변경헀어요.

@changhwan77 changhwan77 merged commit 915d98f into master Feb 9, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant