[업로드] 파일 선택 #105
[업로드] 파일 선택 #105
Conversation
| @@ -27,5 +27,5 @@ android { | |||
| } | |||
|
|
|||
| dependencies { | |||
|
|
|||
| implementation(libs.appcompat) | |||
There was a problem hiding this comment.
registerForActivityResult의 파라미터 넣어줄 때 사용하는 PickVisualMedia랑 ActivityResultContracts가 액티비티를 필요로 해요
| private fun setupSelectThumbnailImage() { | ||
| binding.cvUploadThumbnail.setOnClickListener { | ||
| filePickerLauncher.launch("audio/*") | ||
| } | ||
| } |
There was a problem hiding this comment.
setUpSelectThumbnailImage 인데 filePickerLauncher가 아니라 pickMedia 를 써야할 것 같네용!
| private val pickMedia = registerForActivityResult(PickVisualMedia()) { uri -> | ||
| if (uri == null) return@registerForActivityResult | ||
|
|
||
| viewModel.uploadImage(uri.toFile()) | ||
| } | ||
|
|
||
| private val filePickerLauncher = | ||
| registerForActivityResult(ActivityResultContracts.GetContent()) { uri -> | ||
| if (uri == null) return@registerForActivityResult | ||
|
|
||
| viewModel.uploadAudio(uri.toFile()) | ||
| } | ||
|
|
There was a problem hiding this comment.
2개 변수명 비슷한 형식으로 다시 짓는거 어떨까용??
| android:background="@drawable/ic_camera" | ||
| android:backgroundTint="@color/surface" /> | ||
| android:src="@drawable/ic_camera" | ||
| android:importantForAccessibility="no"/> |
There was a problem hiding this comment.
importantForAccessibility 이거 처음봐서 어떤건지 궁금하네요1!!
There was a problem hiding this comment.
no를 지정하면 스크린 리더가 해당 View를 읽지 않는다는데, 이 아이콘은 장식용 아이콘이라 읽지 않아도 된다고 판단했어.
대신 상위 CardView에 contentDescription을 추가해서 어떤 동작을 수행할 수 있는지 명시해주고 있어
| app:layout_constraintEnd_toEndOf="parent" | ||
| app:layout_constraintStart_toStartOf="parent" | ||
| app:layout_constraintTop_toBottomOf="@id/tb_upload_appbar"> | ||
| app:layout_constraintTop_toBottomOf="@id/tb_upload_appbar" | ||
| android:contentDescription="@string/select_thumbnail"> |
|
고생하셨습니다~ |
|
|
||
| ) : ViewModel() { | ||
|
|
||
| private var uploadedImage: String? = null |
There was a problem hiding this comment.
아직 개발이 덜 되어서, 놔두신것 같은데, nullable 한 string으로 하신 이유가 궁금합니다!
lateinit var 을 쓸 수 도 있을거 같아요
There was a problem hiding this comment.
네이밍은 uploadedImageUrl이 더 적절할 수도 있겠네...
이 프로퍼티는 서버로 썸네일 이미지가 올라갔을 때 채워질 프로퍼티야. 아직 썸네일이 없을 수도 있으니 nullable로 뒀어
There was a problem hiding this comment.
썸네일이 없는 경우는 Blank("") 로 하는 건 어떠신가요???
There was a problem hiding this comment.
변수명은 말씀하신것 처럼 'uploadedImageUrl' 이 더 적절해 보입니다~
Issue
Overview
To Reviewers
#103 이 PR이 먼저 merge된 이후 충돌 해결하고 머지할게