-
Notifications
You must be signed in to change notification settings - Fork 0
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
[PC-225] 매칭 상세 상대방 프로필 확인 페이지 및 수락 Dialog 구현 #19
Conversation
dialogTop: @Composable () -> Unit, | ||
dialogBottom: @Composable () -> Unit, | ||
modifier: Modifier = Modifier, | ||
onDismissRequest: () -> Unit, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- p3 : 저번에 공유해주신 문서처럼 default가 있는 modifier가 람다 밑에 있는게 좋을 것 같은데, 혹시 다른 의도가 있으신건가욥?!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- p3 : 저번에 공유해주신 문서처럼 default가 있는 modifier가 람다 밑에 있는게 좋을 것 같은데, 혹시 다른 의도가 있으신건가욥?!
헉 아닙니다!! 실수에요... 수정해두겠습니다!
.padding(start = 20.dp, end = 20.dp), | ||
) | ||
|
||
AsyncImage( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
호호 제가 compose를 아직 많이 안써봐서 처음 보는 것들이 많네요!! 이건 이미지를 비동기로 불러올 때 쓰는건가욥?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
호호 제가 compose를 아직 많이 안써봐서 처음 보는 것들이 많네요!! 이건 이미지를 비동기로 불러올 때 쓰는건가욥?
서버에서 데이터를 내려오면 해당 이미지에 대한 이미지 주소를 받잖아요,
그러면 그 이미지 주소를 이용해서 Coil이라는 이미지 라이브러리를 사용하여 비동기적으로 이미지를 호출해서 보여주는 역할을 합니다!
내부적으로 Placeholder나 OnError, onLoading 등을 이용해서 로딩 중일 때나 이미지 불러오기를 실패했을 때,
기본 이미지 들을 삽입해줄 수도 있어요!
import com.puzzle.matching.detail.component.ValueTalkHeader | ||
|
||
@Composable | ||
internal fun ValueTalkBody( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
호호 가치관 pick ui 완성하면 분리하려고 했었는데, 대신 해주셔서 감사합니다! ㅎㅎ
한가지 궁금한 점은 현재 저희 프로젝트에 있는 common-ui 모듈의 정의가 궁금합니다!
저는 처음에 common_ui모듈을 보고 nowinandroid와 안드로이드 문서의 ui 모듈과 같은 것이 아닐까 생각했는데, 혹시 제 생각이 맞을까요??
위 문서들을 보고 제가 정의한 ui모듈은 feature 모듈에서 사용하는 ui 컴퍼넌트 들을 저장하는 건데, 태규님이 만들어주신 common_ui 모듈도 같은 정의일까요? 아니면 다른 기준으로 파일을 넣으면 될까요?? 태규님 의견이 궁금합니다!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
제가 문서를 잘못 파악한 것일수도 있어서 혹시 제 정의가 잘못되었다면 말씀해주세요!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
호호 가치관 pick ui 완성하면 분리하려고 했었는데, 대신 해주셔서 감사합니다! ㅎㅎ 한가지 궁금한 점은 현재 저희 프로젝트에 있는 common-ui 모듈의 정의가 궁금합니다!
저는 처음에 common_ui모듈을 보고 nowinandroid와 안드로이드 문서의 ui 모듈과 같은 것이 아닐까 생각했는데, 혹시 제 생각이 맞을까요??
위 문서들을 보고 제가 정의한 ui모듈은 feature 모듈에서 사용하는 ui 컴퍼넌트 들을 저장하는 건데, 태규님이 만들어주신 common_ui 모듈도 같은 정의일까요? 아니면 다른 기준으로 파일을 넣으면 될까요?? 태규님 의견이 궁금합니다!!
NowInAndroid도 아마 common
이랑 ui
모듈로 분리되어 있을텐데요,
common
은 말 그대로 프레임워크 종속성을 가지고 있지 않은 순수 코틀린 언어로만 작성된 모듈입니다.
그렇기 때문에 프레임워크 의존성을 가지고 있지 않은 domain
모듈에서도 해당 모듈을 접근하여 공통 함수를 사용할 수 있구요.
반면, common-ui
는 프레임워크 종속성을 가지고 있는 공통 함수들이 모이는 곳이라고 정의했습니다.
예를 들면 어떤 모듈에서나 사용되는 Modifier 확장함수나, 안드로이드 생명주기를 갖는 코루틴 확장 함수, 디바이스에서 TimeZone이나 현재 위치를 따와야 하는 함수 등등이 위치하는 곳이에요.
그렇기 때문에 클린 아키텍처를 사용한다면 프레임워크 의존성을 가져도 괜찮은 presentaion
, data
레이어 모듈에서 해당 모듈을 접근해도 되는 것이죠!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아하 그렇군요! 그럼 compose와 관련된 코드들만 들어가는 것이 아니였군요! 그럼 공통 component 같은 건 전부 디자인 시스템에서 관리하는 식으로 하면 될까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아하 그렇군요! 그럼 compose와 관련된 코드들만 들어가는 것이 아니였군요! 그럼 공통 component 같은 건 전부 디자인 시스템에서 관리하는 식으로 하면 될까요?
네 맞습니다! NowInAndroid에서 사용하는 ui
모듈하고는 조금 다른 것 같은데,
저희 프로젝트에서는 common-ui
는 designsystem
보다 더 세부적인 모듈이라 designsystem
이 이를 참조할 수 있다고 생각하면 될 것 같습니다!
역할이 조금 다른 것 같아요
NowInAndroid에서는 Designsystem
에는 Material 기반 컴포넌트만 정의하였고, ui
라는 모듈에서 개발자가 정의한 공통 컴포넌트들을 정의하였더라고요.
굳이 이렇게 나눠야되나 하는 마음이 있습니다...!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그렇다면 태규님이 정의하신 기준으로 진행하시죠!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
마지막 페이지 작업해주셔서 감사합니다 😇
import com.puzzle.matching.detail.component.ValueTalkHeader | ||
|
||
@Composable | ||
internal fun ValueTalkBody( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
제가 문서를 잘못 파악한 것일수도 있어서 혹시 제 정의가 잘못되었다면 말씀해주세요!!
1. ⭐️ 변경된 내용
2. 🖼️ 스크린샷(선택)
2024-12-31.7.20.53.mov
4. 📌 이 부분은 꼭 봐주세요!
공식 문서에 나와있는 Blur효과는 Andorid 12 이상에서만 적용이 됩니다.
12 이하 기기를 호환하기 위해 RenderScript를 이용해야 하지만, 이 기능은 Deprecated 되었습니다.
직접 스택 오버플로우나 gpt를 이용해서 만들 수는 있겠지만 내부 동작 원리를 자세히 아는 것도 아니고 성능 상에 문제가 있다고 많이들 말하고 있기에,
Skydoves님이 만드신 라이브러리를 이용하기로 하였습니다.
저희 프로젝트에서는 그냥 블러 처리 딱 그 정도만 필요하기 때문에 해당 라이브러리에서 지원하지 않는 기능이 요구될 때 까지 그냥 사용해도 될 것 같아요.
Cloudy