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

✨ 로그인 Bottom Sheet UI 추가 #7

Closed
wants to merge 2 commits into from
Closed

Conversation

changhwan77
Copy link
Collaborator

Title

로그인 버튼 클릭 시 띄워질 로그인 Bottom Sheet UI 추가

What type of PR is this?

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

Description

로그인 버튼 클릭 시 띄워질 로그인 Bottom Sheet UI 추가했어요.

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

추후 추가할 사항입니다!

  1. 소셜(카카오, 네이버)로그인 기능
  2. 다른 컴포넌트에서도 해당 Sheet를 호출할 수 있도록 수정

@@ -26,7 +26,7 @@ class HttpErrorHandler {
context.go('/home');
}
} else {
TuTiSnackBar.showSnackBar(context, data['message'].toString());
// TuTiSnackBar.showSnackBar(context, data['message'].toString());
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

비로그인 유저가 트티 screen 접속 시 token이 없다고 뜨는 스낵바 잠시 제거했습니다.

constraints:
BoxConstraints(maxWidth: 300.w, maxHeight: 460.h),
child: Padding(
padding: EdgeInsets.fromLTRB(40.w, 20.h, 40.w, 20.h),
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

디자인 시안에 따라 바텀 시트의 크기를 최대 너비 300, 높이 460으로 제한하고
패딩을 왼, 오른쪽 40, 위 아래 20을 줬습니다.

Align(
alignment: Alignment.bottomRight,
child: loginIntroButtonCollection(context)),
],
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. 로그인 안내 텍스트(loginSheetTexts) 2.로그인 버튼 모음(loginIntroButtonCollection) 으로 분리했습니다.

[ layout 관련 ]
Aling 위젯을 활용해 1과 2를 각각 디자인 시안에 맞게 조정했습니다.
그리고 중간에 Spacer 위젯을 활용해 1, 2가 차지하고 남는 공간 전부로 여백을 줬습니다.

{String? foregroundText,
String? imagePath,
Color? backGroundColor,
VoidCallback? onPressed}) {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

추후 로그인 기능을 추구하기 위해 VoidCallBack typedef를 인자로 선언했습니다.

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