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

수정할 사항들 #33

Open
4 of 7 tasks
chanubc opened this issue Nov 6, 2023 · 3 comments
Open
4 of 7 tasks

수정할 사항들 #33

chanubc opened this issue Nov 6, 2023 · 3 comments
Assignees

Comments

@chanubc
Copy link
Collaborator

chanubc commented Nov 6, 2023

📌 수정할 사항

  • datePicker material2에서 material3로 변경

  • 나의정보 화면 프로필 이미지 추가, 프로필 사진 저장은 추후에 구현 한다 해도 프로필 이미지 영역은 필요

  • 나의 정보 화면 디바이더 색상 및 크기 변경

    • 현재색상 -> #F9F9F9
    • 크기 10dp로 변경
  • 검색 화면 editText영역 하단 구분선 제거, 디자인 수정

    • 고정 height 50dp
    • 구분선 제거
    • 좌측 돌아가기 버튼 제거 혹은 검색시 eidtText 내부의 검색 아이콘 대신 돌아가기 버튼 활성화
    • 돌아가기 버튼 제거 후 editText 좌우영역 확대
    • 검색 아이콘 btn_search로 변경
  • 홈 화면 일주일간의 섭취 칼로리 그래프, x축 데이터 오류

  • 분석화면 상단 action bar 제거

  • bottom sheet layout 상단 radius 적용

@yonghanJu
Copy link
Contributor

yonghanJu commented Nov 6, 2023

  1. 나의정보 화면 프로필 이미지 추가, 프로필 사진 저장은 추후에 구현 한다 해도 프로필 이미지 영역은 필요
    -> 로그인이나 소셜 관련 기능이 없어서 이게 굳이 필요할까? 싶어서 안만들긴 했음.
    누구 보여줄 사람도 없으니 사람들이 굳이 사용할 것 같지도 않고
    서버에 넣을 수도 없어서... uri를 사용하면 사진 저장 위치만 바껴도 먹통되고 앱 내부에 저장하기엔 무거워

  2. 나의 정보 화면 디바이더 색상 및 크기 변경 -> 확인이요

  3. 검색 화면 editText영역 하단 구분선 제거, 디자인 수정 -> 디자인 수정은 확인, 근데 돌아가기 버튼이 없으면 디자인적으로 좀 그렇지 않을까해서 넣었던거였음. 아이폰 안드 똑같은 디자인 적용될거라 기대하면 넣는게 맞지 않을까 싶은데 어떻게 생각하나요.

  4. 홈 화면 일주일간의 섭취 칼로리 그래프, x축 데이터 오류 -> 이부분 건든게 일단 없고 8월달(영양통계 화면 성별, 나트륨 표기 오류 수정 #31 ) 커밋 버전을 돌렸는데 똑같기 때문에 형이 한번 봐야할 듯

  5. 분석화면 상단 action bar 제거 -> 확인이요

  6. bottom sheet layout 상단 radius 적용 -> 확인이요

@yonghanJu
Copy link
Contributor

yonghanJu commented Nov 6, 2023

그리고 머터리얼 2 기반으로 앱 디자인 되어있어서 DatePicker 만 3로 사용하는건 어려울듯?? 3로 바꾸면 앱 다 바뀜 홈화면에 있는 Fab 버튼도 네모나게 바뀔거임 (처음에 M3로 프로젝트 만들었는데 형 디자인 해준거보고 M2 디자인이길래 M2로 바꿔서 개발했었음, 다시 올리기 힘들듯)

해봤는데 아래 이미지 처럼 되서 M2유지가 나을 듯
image

@chanubc
Copy link
Collaborator Author

chanubc commented Nov 8, 2023

  1. 이 부분에서 주간 칼로리정보가 6개 아래면 null 처리 되있는거 같은데
    예를들어 오늘이 4일인경우 날짜가 총 6개가 안되니까 null처리 되는거 같아
    addAll(
    (_uiState.value.weeklyCaloriesUiState as? WeeklyCaloriesUiState.Avail)
    ?.weeklyCaloriesList?.subList(1, 7)
    ?: List(6) { Pair(0, 0) },
    )

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

No branches or pull requests

2 participants