Feature/#47 design system alert #51

lsakee · 2023-12-11T11:16:46Z

📌 PR 요약

🌱 작업한 내용

다이얼로그 component
toast view component

🌱 PR 포인트

다이얼로그 컴포넌트 뷰 확인해주세요
토스트 뷰 확인 해주세요
네이밍 컨벤션 확인 해주세요
🌱 PR 포인트2
추가 반영 사항 - pr 리뷰 반영 하였습니다
버튼 컴포넌트 속성 1개 바꼇습니다

📸 스크린샷

|스크린샷|

|:--:| |파일첨부바람|

📮 관련 이슈

Resolved: [Feat] Feature Modal #47

jinukeu

전체적으로 타이포그래피, 컬러 적용 부탁드릴게요!

jinukeu · 2023-12-11T11:37:42Z

core/designsystem/src/main/java/com/suwiki/core/designsystem/component/dialog/SuwikiDialog.kt

+@Composable
+fun basicAlertDialog(
+  onDismissRequest: () -> Unit,
+  properties: DialogProperties = DialogProperties(),
+  content: @Composable () -> Unit,
+) {
+  Dialog(
+    onDismissRequest = onDismissRequest,
+    properties = properties,
+  ) {
+    content()
+  }
+}


basicAlertDialog랑 Dialog랑 완전히 동일한데 basic으로 만든 이유가 궁금해요~

https://sungbin.land/jetpack-compose-%EC%99%84%EB%B2%BD%ED%95%9C-%EC%BB%A4%EC%8A%A4%ED%85%80-%EB%8B%A4%EC%9D%B4%EC%96%BC%EB%A1%9C%EA%B7%B8-%EB%A7%8C%EB%93%A4%EA%B8%B0-79aab4c3023e
요블로그를 참고하며 구현하였는데 현재까지는 완전히 동일하니 빼도좋을거 같지만 Dialog가 동일하지 않을 수도 있다고 생각하였습니다!

음 근데 삭님이 만드신 basicAlertDialog도 거의 백지에 가까운 컴포저블이고 Dialog도 백지에 가까운 컴포저블이어서요.
basicAlertDialog에 틀이 잡혀있는게 아니기에 Dialog와 동일한 컴포저블이라 생각해요.
이런 이유로 다른 형태의 Dialog가 나온다해도 basicAlertDialog을 사용해서 구현하는 것과 Dialog를 사용해서 구현하는 것과 차이가 없다 생각합니다.

또 전체 디자인이 이미 나온 상태이기에 새로운 Dialog가 나올 일은 없어보입니당

BaseActivity와 Activity가 아예 동일하다면 Activity만 사용하는게 좋은 것 처럼요!

넴 감사합니다! 수정해서 다시 올리겠습니다!

jinukeu · 2023-12-11T11:39:13Z

core/designsystem/src/main/java/com/suwiki/core/designsystem/component/dialog/SuwikiDialog.kt

+  onConfirmClick: () -> Unit,
+  onDismissClicked: () -> Unit,


Clicked랑 Click 차이가 뭔가요!?

onClickConfirm, onClickDismiss로 순서 바꿔주세요~!

jinukeu · 2023-12-11T11:41:12Z

core/designsystem/src/main/java/com/suwiki/core/designsystem/component/toast/SuwikiToastView.kt

+        ),
+      color = Color.Transparent,
+    ) {
+      // 아이콘이 추가 될수 있을거 같아 일단 Row로 해두었습니다.


dry하게 가시죵 Row 제거해주세요

jinukeu · 2023-12-11T11:42:45Z

core/designsystem/src/main/java/com/suwiki/core/designsystem/component/dialog/SuwikiDialog.kt

+            SuwikiContainedGreyButtonSmall(
+              onClick = onDismissClicked,
+              text = dismissButtonText,
+              modifier = Modifier.padding(2.dp),
+            )


요거 피그마랑 다른거같은데 확인 가능할까요?

SuwikiContainedGreyButtonSmall 버튼과 SuwikiContainedButtonSmall 버튼의 패딩 차이가 있어 저렇게 구현하였습니다!

아하 PR에 첨부한 사진을 보니 백그라운드 컬러가 들어갔더라구요!?

jinukeu · 2023-12-11T13:08:49Z

core/designsystem/src/main/java/com/suwiki/core/designsystem/component/toast/SuwikiToastView.kt

+
+@Composable
+fun SuwikiToastView(
+  messageTxt: String,


messageText로 변경 가능할까요!?

jinukeu

💯 👍

BEEEAM-J · 2023-12-11T16:02:46Z

core/designsystem/src/main/java/com/suwiki/core/designsystem/component/toast/SuwikiToastView.kt

+          color = Gray95,
+          shape = RoundedCornerShape(25.dp),
+        ),
+      color = Color.Transparent,


Surface 많이 잘 몰라서 그런데 color = Color.Transparent 이걸 따로 정의한 이유가 뭔가요???
background랑 무슨 차이인지 잘 모르겠어서..

Surface가 투명한 배경을 가지도록 필요한 코드입니다.
백그라운드 설정으로 가려진 속성이 보이지 않는 것을 방지하는 코드 입니다.
저코드 지우면 토스트 뷰는 보이지 않을 것입니다!

BEEEAM-J

코드 아주 깔끔하네유 👍👍
보기 아주 좋습니다

lsakee added 4 commits December 11, 2023 19:35

feat : #47 Suwiki Dialog

9752ab5

feat : #47 Suwiki Toast View

52e2977

refactor : #47 Component Surface 수정

119a786

chore : #47 import 제거

8cadfcd

lsakee added the feature label Dec 11, 2023

lsakee requested review from jinukeu and BEEEAM-J December 11, 2023 11:16

jinukeu requested changes Dec 11, 2023

View reviewed changes

lsakee added 2 commits December 11, 2023 21:25

refactor : #47 클릭 네이밍 / 불필요 레이아웃 제거

d00c11b

chore : #47 ktlint

822972c

jinukeu reviewed Dec 11, 2023

View reviewed changes

lsakee added 2 commits December 12, 2023 00:19

refactor : #47 Color , Typography set

55432de

chore : #47 ktlint

6737822

lsakee requested a review from jinukeu December 11, 2023 15:25

jinukeu approved these changes Dec 11, 2023

View reviewed changes

BEEEAM-J reviewed Dec 11, 2023

View reviewed changes

BEEEAM-J approved these changes Dec 11, 2023

View reviewed changes

lsakee merged commit 17944e8 into develop Dec 11, 2023
1 check passed

jinukeu assigned lsakee Dec 16, 2023

jinukeu deleted the feature/#47-design-system-alert branch December 20, 2023 13:19

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/#47 design system alert #51

Feature/#47 design system alert #51

lsakee commented Dec 11, 2023 •

edited

Loading

jinukeu left a comment

jinukeu Dec 11, 2023

lsakee Dec 11, 2023

jinukeu Dec 11, 2023

lsakee Dec 11, 2023

jinukeu Dec 11, 2023

jinukeu Dec 11, 2023

jinukeu Dec 11, 2023

lsakee Dec 11, 2023

jinukeu Dec 11, 2023

jinukeu Dec 11, 2023

jinukeu left a comment

BEEEAM-J Dec 11, 2023

lsakee Dec 12, 2023

BEEEAM-J left a comment

Feature/#47 design system alert #51

Feature/#47 design system alert #51

Conversation

lsakee commented Dec 11, 2023 • edited Loading

📌 PR 요약

📸 스크린샷

📮 관련 이슈

jinukeu left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jinukeu left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

BEEEAM-J left a comment

Choose a reason for hiding this comment

lsakee commented Dec 11, 2023 •

edited

Loading