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

대시보드 디자인 개선안 반영 #74

Merged
merged 19 commits into from
Feb 19, 2025
Merged

Conversation

nohjunh
Copy link
Member

@nohjunh nohjunh commented Feb 17, 2025

대시보드 디자인 개선안 반영입니다.

  • 대시보드 바텀시트 컬러 white -> gray50으로 변경
  • DobeDobeCheckBox에서 쓰이는 IconButton은 내부적으로 minimumInteractiveComponentSize()으로 인해, 최소 사이즈가 정해져있음. 그러나, 이번 개선안에서는 아이콘 사이즈를 36으로 가져가길 원하셔서, 최소 사이즈를 무시하기 위해 커스텀하여 수정
  • heading1 typo 일부 값 변경
  • dropShaodw Modifier Ext 추가
  • 기존 대시보드 isExpanded 판단 조건이 모호하여, 확실히 판단하고자 isNotPartiallyExpanded으로 수정
    • 기존처럼 targetValue == SheetValue.isExpanded로 보게 되면, 바텀시트가 완전히 확장이 되지 않더라도, 확장으로의 진행이라면 isExpanded가 됨.
    • currentValue는 완전히 확장/축소가 된 상태여야지 Expaned/PartiallyExpanded가 됨.
    • 그러나, 요구사항에서 필요한 건, PartiallyExpanded가 아닌 상태. 즉, 확장이 되고 있거나/확장이 완료된 상태를 알아야 함.
    • isNotPartiallyExpanded로 의도한 상태를 파악할 수 있도록 수정
  • 버블에서 "~~~해낼 거야!" 부분 제거
  • 그 외 캐릭터/비전보드 등 단순 사이즈 변경
image image

@nohjunh nohjunh self-assigned this Feb 17, 2025
@nohjunh nohjunh requested a review from murjune February 17, 2025 15:05
Copy link
Member

@murjune murjune left a comment

Choose a reason for hiding this comment

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

작업하느라 고생하셨습니다!
몇 가지 궁금한 점만 코멘트로 남겨봤습니다~

@nohjunh nohjunh force-pushed the feat/dashboard-design-v2.0 branch from 54b4b46 to d660fa1 Compare February 19, 2025 13:52
Copy link
Member

@murjune murjune left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~

@nohjunh nohjunh merged commit b455562 into develop Feb 19, 2025
1 check passed
@nohjunh nohjunh deleted the feat/dashboard-design-v2.0 branch February 19, 2025 14:25
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.

2 participants