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

네비게이션바 구현 #21

Merged
merged 3 commits into from
Aug 8, 2024
Merged

Conversation

bywindow
Copy link
Collaborator

@bywindow bywindow commented Aug 7, 2024

  • TopBar 구현
  • 온보딩, 미션생성, 초대코드 입력 화면에 TopBar 추가
  • 다크모드일 때 statusbar 텍스트가 보이지 않던 버그 수정

@bywindow bywindow requested a review from eshc123 August 7, 2024 17:27
@bywindow bywindow self-assigned this Aug 7, 2024
Copy link
Member

@eshc123 eshc123 left a comment

Choose a reason for hiding this comment

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

고생많았어! CompositionLocal 관련해서 코멘트 남겨두긴 했는데 궁금한 걸 물어본거라 Approve 했어!

leftActionButtons: @Composable (() -> Unit)? = null,
rightActionButtons: @Composable () -> Unit = {},
) {
CompositionLocalProvider(LocalContentColor provides contentColor) {
Copy link
Member

Choose a reason for hiding this comment

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

나는 아직 CompositionLocal을 많이 안 써봐서 잘 몰라서 물어보는건데..! 혹시 CompositionLocal을 사용한 이유가 어떻게 될까?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

나도 이번에 새롭게 알게 됐는데 CompositionLocalProvider 사용하면 이 안에 있는 컴포넌트는 모두 provider하는 특성을 유지한다고 하더라고! 그래서 네비게이션 같은 곳에서 백그라운드 컬러 줄 떄 사용되는 것 같아!

@bywindow bywindow merged commit 69ef97c into dev Aug 8, 2024
1 check passed
@bywindow bywindow deleted the feature/implement-topbar-component branch August 8, 2024 11:19
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