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

[ADD/#16] Button / 컴포넌트 구현 #21

Merged
merged 22 commits into from
Jul 9, 2024
Merged

Conversation

leeeyubin
Copy link
Member

@leeeyubin leeeyubin commented Jul 7, 2024

⛳️ Work Description

  • pressed 일 때의 상태 추가
  • TerningBasicButton 구현
  • RectangleButton 구현
  • RoundButton 구현
  • TerninMain2 색상 추가
  • core > designsystem 패키징 수정 & 프리뷰 라이브러리 추가

📸 Screenshot

1️⃣RectangleButton

  • 비활성화 상태

  • 활성화 상태

  • pressed 상태

2️⃣RoundButton

  • 비활성화 상태

  • 활성화 상태

  • pressed 상태

📢 To Reviewers

  • 그리구 헷갈릴까봐 테마 함수명 TerningTheme -> TerningPointTheme으로 바꿨어요!
    타이포 사용할 땐 그대로 TerningTheme으로 접근하시면 됩니당
  • 근데 마지막에 의존성 그래프는 왜 들어갔는지 모르겠음,,, 들어간 김에 다들 한번씩 확인해요🤣🤣

@leeeyubin leeeyubin added ADD ➕ 부수적인 코드 추가 및 라이브러리 추가, 새로운 파일 생성 유빈💙 유빈 labels Jul 7, 2024
@leeeyubin leeeyubin added this to the 1차 스프린트 UI 작업 milestone Jul 7, 2024
@leeeyubin leeeyubin self-assigned this Jul 7, 2024
@leeeyubin leeeyubin changed the title [ADD/#16] Button 컴포넌트 구현 [ADD/#16] Button / 컴포넌트 구현 Jul 7, 2024
Copy link
Member

@boiledEgg-s boiledEgg-s left a comment

Choose a reason for hiding this comment

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

컴포넌트들을 뚝딱뚝딱 구현하는 터닝 안드원들 신기하고 대단하네요,,!!
역시 이번 피알도 엄청납니당,, 최고!!!!

Comment on lines +43 to +45
val interactionSource = remember { MutableInteractionSource() }
val isPressed by interactionSource.collectIsPressedAsState()
val backgroundColor = if (isPressed) TerningMain2 else TerningMain
Copy link
Member

Choose a reason for hiding this comment

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

버튼 색상 변경 방법 신기하네요!!
InteractionSource를 알아보다가 말았는데 여기서 배우고 갑니다!!

Copy link
Member

Choose a reason for hiding this comment

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

오호!! 저도 새로운거 알아갑니다~~!!

Copy link
Member

@Hyobeen-Park Hyobeen-Park left a comment

Choose a reason for hiding this comment

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

진짜 너무 최고고,,, 너무 멋있고,,, 너무 짱이고,,, 정말이지 최고에요!! 수고하셨습니당❤️

Comment on lines +43 to +45
val interactionSource = remember { MutableInteractionSource() }
val isPressed by interactionSource.collectIsPressedAsState()
val backgroundColor = if (isPressed) TerningMain2 else TerningMain
Copy link
Member

Choose a reason for hiding this comment

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

오호!! 저도 새로운거 알아갑니다~~!!

@@ -33,7 +33,7 @@ fun ProvideTerningTypography(typography: TerningTypography, content: @Composable
}

@Composable
fun TerningTheme(
fun TerningPointTheme(
Copy link
Member

Choose a reason for hiding this comment

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

우왕 완전 센스쟁이!!

val isPressed by interactionSource.collectIsPressedAsState()
val backgroundColor = if (isPressed) TerningMain2 else TerningMain

CompositionLocalProvider(LocalRippleTheme provides NoRippleTheme) {
Copy link
Member

Choose a reason for hiding this comment

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

궁금한거 있습니다!!! 이렇게 선언만 해두면 바로 ripple이 사라지는건가요??????

Copy link
Member Author

Choose a reason for hiding this comment

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

네! 기존에 있던 LocalRippleTheme을 NoRippleTheme으로 바꿔준 겁니당

Copy link
Contributor

@arinming arinming left a comment

Choose a reason for hiding this comment

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

베이직 버튼 요긴하게 잘 쓰겠습니당 ㅜㅜ!!

) {
Column(
modifier = Modifier
.fillMaxSize(),
.wrapContentHeight() // Modifier 변경
Copy link
Contributor

Choose a reason for hiding this comment

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

여기 주석은 일부러 남겨두신 걸까욤?

Copy link
Member Author

Choose a reason for hiding this comment

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

지우는 걸 깜빡했네요!😂 수정했습니당

Comment on lines +13 to +18
override fun rippleAlpha(): RippleAlpha = RippleAlpha(
draggedAlpha = 0.0f,
focusedAlpha = 0.0f,
hoveredAlpha = 0.0f,
pressedAlpha = 0.0f
)
Copy link
Contributor

Choose a reason for hiding this comment

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

리플테마 적용 감사해용 🥹

@leeeyubin leeeyubin merged commit f580d96 into develop Jul 9, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ADD ➕ 부수적인 코드 추가 및 라이브러리 추가, 새로운 파일 생성 유빈💙 유빈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ADD] Button 컴포넌트 구현
4 participants