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

[Feat] 밈 등록 디자인 구현 #249

Merged
merged 19 commits into from
Sep 27, 2024
Merged

[Feat] 밈 등록 디자인 구현 #249

merged 19 commits into from
Sep 27, 2024

Conversation

ze-zeh
Copy link
Collaborator

@ze-zeh ze-zeh commented Sep 26, 2024

Issue

작업 내역 (Required)

  • 밈 등록 디자인 구현

Review Point (Required)

  • none

Screenshot

1.mp4

관련 링크

  • none

Copy link
Collaborator

@EvergreenTree97 EvergreenTree97 left a comment

Choose a reason for hiding this comment

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

UI 권위자 ㄷㄷ

text = "Text",
enabled = false,
)
}
}

@Composable
@Preview
fun FarmemeMediumChipPreview() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Preview는 private로 숨겨주면 좋을것같아~ ChipPreview가 자동완성에 뜨면 곤란곤란

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

과거의 흔적... 전수조사해봐야겟다

hasImage = false,
)
}
item { RegisterInputArea() }
Copy link
Collaborator

Choose a reason for hiding this comment

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

요기 깔끔히 잘나눴네유

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

후후~

}
},
bottomBar = {
RegisterButton(
Copy link
Collaborator

Choose a reason for hiding this comment

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

버튼 공중에 떠있는건 스펙인가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

예압,, 근데 bottombar에 넣는게 괜찮은지 궁금해요~

Copy link
Collaborator

@hj1115hj hj1115hj left a comment

Choose a reason for hiding this comment

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

Ui gpt인뎅.. 왜케잘하누! 깔끔

@Composable
internal fun RegisterCategoryContent(
modifier: Modifier = Modifier,
uiModel: RegisterCategoryUiModel,
Copy link
Collaborator

Choose a reason for hiding this comment

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

오 uiModel 네이밍 좋당

@Composable
internal fun RegisterCategoryChips(
modifier: Modifier = Modifier,
keywords: ImmutableList<String>,
Copy link
Collaborator

Choose a reason for hiding this comment

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

꼼꼼하세용!

# Conflicts:
#	core/designsystem/src/main/kotlin/team/ppac/designsystem/foundation/Icon.kt
@ze-zeh ze-zeh merged commit bb9a20a into develop Sep 27, 2024
@ze-zeh ze-zeh deleted the feat/register-ui branch September 27, 2024 04:34
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.

밈 등록 Register 디자인 구현
3 participants