Feature/360 입장 확인 고도화 #363
Conversation
Test Results 9 files 9 suites 0s ⏱️ Results for commit 29514c9. ♻️ This comment has been updated with latest results. |
|
전면 카메라 전환 시연 영상은 부끄러워서 안 넣었어 |
| @@ -40,6 +47,21 @@ class QrScanActivity : ComponentActivity() { | |||
| private val callback = BarcodeCallback { result: BarcodeResult -> | |||
| result.text ?: return@BarcodeCallback | |||
| viewModel.scan(result.text) | |||
|
|
|||
| val vibrator = | |||
There was a problem hiding this comment.
추후에 진동 요구사항이 또 들어오면 확장으로 만들어도 좋을 듯
| private fun switchCamera() { | ||
| barcodeView.pause() | ||
| isBackCamera = !isBackCamera | ||
| barcodeView.cameraSettings.requestedCameraId = if (isBackCamera) { |
There was a problem hiding this comment.
분명 전/후면 카메라 중 하나가 없는 기기가 있을텐데, 그럴 땐 어떻게 동작해?
There was a problem hiding this comment.
맞아 그 부분이 문제긴 해.
사실 xzing 라이브러리 자체가 camera2보다 이전인 camera 기반으로 동작하는 레거시이긴 하더라고...
그래서 mlkit에 있는 바코드 스캐너 사용해 보려 했는데 급하게 작성하느라... 허허...
전면 카메라 식별이 가능한 거 같은데, 전면 카메라 없으면 버튼을 없애는 식으로 처리해 볼게
There was a problem hiding this comment.
레거시이긴 하지만 오히려 Camera2 를 지원하지 않는 기기들이 있어서 호환성은 아이러니하게 Camera1이 더 좋은 상황...
카메라 식별해서 버튼 없애는 건 좋은 것 같아!
There was a problem hiding this comment.
아 마져 camerax가 요구 api가 되게 높았던 거 같다
There was a problem hiding this comment.
근데 xzing 도 사용하는데 무리없어서 마이그레이션이 급한 건은 아닌 듯
| @@ -24,6 +30,7 @@ import kotlinx.coroutines.launch | |||
|
|
|||
| @AndroidEntryPoint | |||
| class QrScanActivity : ComponentActivity() { | |||
| var isBackCamera = true | |||
There was a problem hiding this comment.
이 속성을 여기서 관리했을 때 카메라 방향 싱크가 안 맞는 문제는 없을까?
|
|
||
| launch { | ||
| borderColor = color | ||
| delay(4000L) |
There was a problem hiding this comment.
4초인 이유가 있을까? QR 스캔 인터벌이 1초라서 4초 이내에 새로운 실패가 발생하면 4초가 보장이 안 될 것 같은데
예를 들어
0초: 실패하여 빨강으로 표시
3초 후: 새로운 QR 스캔 - 실패하여 노랑으로 표시
1초 후: 맨 처음 실패 이후 4초가 지나 스트로크 사라짐
그럼 유저가 봤을 땐 빨강으로 3초, 노랑으로 1초만에 스트로크가 없어져서 이상하게 느껴질 것 같은 걱정이 드네
There was a problem hiding this comment.
이거는 스낵바 시간에 맞춰서 4초로 한 거긴 해
나도 만들면서 그생각 하긴 했는데 1초로 하는 게 나을까?
There was a problem hiding this comment.
이거는 혜선이랑 확실히 맞추고 가는게 좋을 듯!
인식 후 1초동안 멈추는게 에러 표시도 1초인건지,
그게 아니라면 새로운 결과가 인식되면 이전 job 은 cancel 해주는 게 좋을 듯!
| leadingIcon = { | ||
| snackbarIconId?.let { | ||
| CircleBgIcon( | ||
| painter = painterResource(it), |
There was a problem hiding this comment.
이거 vector 이미지라면 ImageVector.vectorResource 인가 이거 쓰는게 Stable 이라서 더 좋을 듯!
There was a problem hiding this comment.
수정하면서 같이 바꿀게
수정한 부분은 아닌데 들여쓰기 때문에 diff에 들어간 듯 ㅋㅋㅋㅋ
근데 inner padding부분 수동으로 offset 변경해 줘도 되는지 모르겠넹...
구글이 의도한 동작인가... 버그인가...
Issue
작업 내용
KakaoTalk_Video_2025-01-09-22-50-26.mp4