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

taxiDialog 리팩토링 #71

Merged
merged 22 commits into from
Sep 11, 2023
Merged

taxiDialog 리팩토링 #71

merged 22 commits into from
Sep 11, 2023

Conversation

ybmin
Copy link
Contributor

@ybmin ybmin commented Aug 23, 2023

Summary

It closes #70

@ybmin ybmin added enhancement New feature or request working labels Aug 23, 2023
@ybmin ybmin self-assigned this Aug 23, 2023
@ybmin ybmin linked an issue Aug 23, 2023 that may be closed by this pull request
@ybmin ybmin changed the title Add: theme data 추가 taxiDialog 리팩토링 Aug 23, 2023
@ybmin
Copy link
Contributor Author

ybmin commented Aug 29, 2023

추가적으로 해결한 문제점: 확인 버튼 하단 패딩이 맞지 않습니다.

@ybmin
Copy link
Contributor Author

ybmin commented Aug 29, 2023

Dialog 리팩토링

☑️ 표시는 현재 커밋에 반영된 부분입니다.

I. 테마 코드 분리

  1. ☑️ 제목 강조(볼드)
style: GoogleFonts.roboto(
                            textStyle: const TextStyle(
                                color: Color(0xFF323232),
                                fontSize: 22,
                                fontWeight: FontWeight.bold)),
  1. ☑️ 일반 제목

    style: TextStyle(color: Color(0xFF323232), fontSize: 22, fontWeight: FontWeight.normal)),

  2. ☑️ 일반 제목 (보라색)

    style: TextStyle(color: Color(0xFF6E3678),fontSize: 22, fontWeight: FontWeight.bold)),

  3. ☑️ 설명글

style: GoogleFonts.roboto(
                          textStyle: const TextStyle(
                              color: Color(0xFF888888),
                              fontSize: 12,
                              fontWeight: FontWeight.bold))),

과거: 현재:
Untitled 2

  • 현 택시 피그마 디자인 기준:
    Untitled 3
    Untitled 4

  • 발견된 문제점: 확인 버튼 하단 패딩이 맞지 않습니다. (해결 완료)

    • 추가: “정상적인 사용을 위해 앱을 업데이트 해주세요.”나 서버 연결 오류 시 발생하는 설명 글의 “일시적인 오류일 수 있습니다.”와 같은 설명 텍스트에 노란 밑줄이 의도된 사용인가요?

    • 패딩 개선:

      • 개정1
        Untitled 5

      • 개정2
        Untitled 6

  • primaryColor가 blue로 설정되어있어 main.dart의 색상을 참고해 0xFF6E3678을 기준으로 작업했습니다.

II. 공통 설정 상수화

constants와 theme.dart를 활용하여 공통되는 설정과 관련된 내용을 정리하고 있습니다.

III. (추가 진행중)

@14KGun
Copy link
Member

14KGun commented Aug 31, 2023

@ybmin 제가 확인이 늦었네요ㅜㅜ 예리한 지적 감사합니다.

테마 관련 코드들을 통합하시고 계신 것 같은데 다음을 참고해 주시면 좋을 것 같습니다.

  1. 택시의 메인 폰트는 NanumSquare 입니다. (피그마 폰트는 메인 폰트가 아니에요 ㅜ) 제가 디자이너는 아니라 모르는데 검색해보니 네이버의 폰트라고 하네요. 폰트 파일은 taxi-front/src/static/fonts 에서 확인하실 수 있어요.
  2. 버튼 사이의 간격이 넓어 보입니다. 버튼 사이 간격 권장 값은 10px 입니닷
  3. 테마 관련하여 taxi-front는 theme.ts 이러한 설정 값들을 쓰고 있습니다 ㅎ

Comment on lines 49 to 47
style: GoogleFonts.roboto(
textStyle: const TextStyle(
color: Color(0xFFC8C8C8),
fontSize: 13,
fontWeight: FontWeight.normal))),
style: Theme.of(context).textTheme.labelMedium),
Copy link
Member

Choose a reason for hiding this comment

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

👍 깔끔해지네요 ㅎ

Copy link
Collaborator

@happycastle114 happycastle114 left a comment

Choose a reason for hiding this comment

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

깔끔해졌네요 !

@ybmin ybmin requested a review from 14KGun September 7, 2023 18:09
@ybmin ybmin marked this pull request as ready for review September 7, 2023 18:09
Copy link
Member

@14KGun 14KGun left a comment

Choose a reason for hiding this comment

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

LGTM !

Copy link
Contributor

@sboh1214 sboh1214 left a comment

Choose a reason for hiding this comment

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

Looks good to me

@happycastle114 happycastle114 merged commit f0276e4 into main Sep 11, 2023
2 checks passed
@happycastle114 happycastle114 deleted the #70-taxidialog-리팩토링 branch September 11, 2023 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

taxiDialog 리팩토링
4 participants