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

워키TopBar 구현 #39

Merged
merged 3 commits into from
Sep 14, 2023
Merged

워키TopBar 구현 #39

merged 3 commits into from
Sep 14, 2023

Conversation

soopeach
Copy link
Member

😎 작업 내용

  • 재활용 가능한 WalkieTopBar 구현

🧐 변경된 내용

  • 변경된 내용을 작성해주세요.

🥳 동작 화면

  • 커뮤니티 메인
Screenshot 2023-09-12 at 3 12 25 PM
  • 챌린지 메인
Screenshot 2023-09-12 at 3 12 55 PM

🤯 이슈 번호

  • 이슈 없음

🥲 비고

  • 중앙에만 아이템을 배정하고 싶을 경우 leftContent, rightContent는 Spacer와 같은 빈 컴포저블로 위치를 할당시키면 됨.

@soopeach soopeach added the ASAP 바로 머지해주세요! label Sep 12, 2023
@soopeach soopeach requested a review from yonghanJu September 12, 2023 06:14
@soopeach soopeach self-assigned this Sep 12, 2023
Comment on lines +14 to +23
fun WalkieTopBar(
leftContent: @Composable (() -> Unit)? = null,
middleContent: @Composable (() -> Unit)? = null,
rightContent: @Composable (() -> Unit)? = null,
) {
Row(
modifier = Modifier
.fillMaxWidth()
.height(54.dp)
.padding(20.dp, 13.dp),
Copy link
Contributor

Choose a reason for hiding this comment

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

매게변수에 modifier를 추가적으로 받게하는건 어떻게 생각하시나요??

@yonghanJu yonghanJu merged commit 6921f73 into compose/develop Sep 14, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ASAP 바로 머지해주세요!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants