Skip to content

Conversation

@impri2
Copy link
Collaborator

@impri2 impri2 commented May 21, 2025

작업 요약

식단 뷰 UI 개편

관련 이슈

closes #5

변경 항목 (모두 체크)

  • 기능 추가
  • 기능 개선
  • 버그 수정
  • 디자인 변경
  • 코드에 영향 주지 않는 변경(들여쓰기, 공백, 변수명 변경 등)
  • 리팩토링
  • 파일명/폴더명 수정
  • 파일/폴더 삭제
  • 빌드 설정 수정
  • 패키지 매니저 수정
  • 문서 수정
  • 테스팅 코드

변경 내용 상세

  • 메인 화면 UI 개편
  • 달력 UI 개편
  • 식당정보 UI 개편
  • 필터 선택 화면 UI 개편

변경 후 스크린샷/녹화 (필요 시)

향후 개선 필요 사항

"식단 정보가 없습니다" 문구, 아침/점심/저녁 선택 버튼 텍스트가 figma에서 font system 미적용되어 있어 적용되면 수정 예정

리뷰 시 참고할 사항 및 집중 리뷰 요청 부분

개발자 체크리스트

  • 팀 코딩 컨벤션에 맞게 작성(README 혹은 Notion 확인)
  • 변경 사항 테스팅 완료 (버그수정 혹은 기능추가/개선)
  • client id와 같은 노출되면 안 되는 정보는 노출 X (커밋기록에 나타나는 것도 X)

리뷰어 체크리스트

  • 기능 검증(테스트)
  • 코드 가독성, 중복 여부
  • 성능 저하 우려 부분 존재 여부
  • 예외 처리 및 오류 대응 여부
  • 코딩 컨벤션 부합 여부

impri2 and others added 27 commits May 19, 2025 15:26
* chore: apply standardized Info.plist variable syntax to KAKAO_APP_KEY

* refactor: centralize config.plist access with type-safe Config singleton

* chore: apply standardized Info.plist variable syntax to NAVER_MAP_CLIENT_ID
* bugfix: resolve account removal error

* fix: add 204 to empty response codes for community post and comment removal

* fix: remove userdefaults after remove account success

* refactor: change userdefaults set nil to removeobject

---------

Co-authored-by: impri2 <[email protected]>
* chore: delete unused color assets & add dark mode gray color

* chore: delete past gray color assets

* chore: replace usage of old gray colors

* add: add foundation/orange colors

* chore: replace usage of old orange colors

* chore: color asset foldering

* chore: delete old main color

* chore: replace old main color with Orange500

* fix: textView color error

* fix: replace restuarantView main color to Orange500

* chore: enable dark mode

* chore: add SementicColor

* chore: add missing colors

---------

Co-authored-by: h9kwon <[email protected]>
* chore: add namespace to color assets

* chore: remove unnecessary prefix from color asset names

* fix: typo in color assets name
@impri2 impri2 marked this pull request as ready for review July 28, 2025 08:30
JH713
JH713 previously approved these changes Aug 3, 2025
@impri2 impri2 dismissed JH713’s stale review August 3, 2025 09:23

The merge-base changed after approval.

Copy link
Collaborator

@JH713 JH713 left a comment

Choose a reason for hiding this comment

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

확인 완료!

@impri2 impri2 merged commit 62b2ef3 into main Aug 3, 2025
impri2 added a commit that referenced this pull request Aug 3, 2025
impri2 added a commit that referenced this pull request Aug 3, 2025
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.

즐겨찾기/메인 탭 UI 개편

5 participants