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.

Sign up for GitHub

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

Jump to bottom

SnackBar 구현 #14

Open
wants to merge 26 commits into
base: main
Choose a base branch
from
Open

SnackBar 구현 #14

wants to merge 26 commits into from

Conversation

leeeyubin
Copy link
Member

@leeeyubin leeeyubin commented Sep 23, 2024 •
edited
Loading

SnackBar 구현했습니다.

  • 스낵바 올라오고, 내려가는 애니메이션 구현
  • 정보성 스낵바 5초 후, 자동으로 사라지는 기능
  • 정보성 스낵바 아래로 스와이프 시, 사라지는 기능
  • 에러 스낵바 엑스 아이콘 클릭 시, 사라지는 기능
  • 두 줄 넘어갈 시, 단어 단위의 개행
image

기본 기능

Screen_Recording_20241031_110536_Handy-Android.mp4

스와이프 기능

drag.mp4

@leeeyubin leeeyubin added the enhancement New feature or request label Sep 23, 2024
@leeeyubin leeeyubin self-assigned this Sep 23, 2024
kangyuri1114
kangyuri1114 reviewed Sep 23, 2024
View reviewed changes
Copy link
Member

@kangyuri1114 kangyuri1114 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 수정사항이 있어서 확인 부탁드려요!

compose/src/main/kotlin/com/yourssu/handy/compose/SnackBar.kt Outdated
) {
Column(
modifier = modifier
.padding(horizontal = 16.dp)
Copy link
Member

Choose a reason for hiding this comment

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

padding 이 두번 들어간 것 같습니다!

compose/src/main/kotlin/com/yourssu/handy/compose/SnackBar.kt Outdated Show resolved Hide resolved
compose/src/main/kotlin/com/yourssu/handy/compose/SnackBar.kt
) {
Row(
modifier = modifier
.padding(horizontal = 16.dp)
Copy link
Member

Choose a reason for hiding this comment

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

여기 padding도용

compose/src/main/kotlin/com/yourssu/handy/compose/SnackBar.kt Show resolved Hide resolved
compose/src/main/res/drawable/ic_alert_triangle.xml Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kangyuri1114 kangyuri1114 kangyuri1114 left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@leeeyubin leeeyubin

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@leeeyubin @kangyuri1114