-
Notifications
You must be signed in to change notification settings - Fork 1
인증샷 화면 QA 1차 반영 #125
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
Merged
Merged
인증샷 화면 QA 1차 반영 #125
Changes from 2 commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
35b374d
✨ Feat: CommentTextField.kt 패딩 수정
chanho0908 7cbd7bd
✨ Feat: `CommentTextField` `onHeightMeasured` 콜백을 추가하여 컴포넌트 높이 변화를 감지…
chanho0908 08802c1
♻️ Refactor: `CameraControlBar` 내부 레이아웃 구조 개선
chanho0908 153a8f5
♻️ Refactor: `AppRoundButton` 테두리 두께 변경 (1.dp -> 1.6.dp)
chanho0908 a203dd7
✨ Feat: `AppRoundButton` 공용 컴포넌트 고도화 및 UI 일관성 적용
chanho0908 f8f2e52
♻️ Refactor: "인증샷 찍기"에서 "업로드하기"로 변경하여 사용자 가이드 문구 수정
chanho0908 47611dd
♻️ Refactor: `CommentTextField` 배경 그리기 방식 개선 및 리팩토링`
chanho0908 a24d38d
♻️ Refactor: 인증샷 편집 화면 재촬영 버튼 높이 수정
chanho0908 07b2fcb
✨ Feat: `AppRoundButton` 롱 버튼 프리뷰 추가
chanho0908 26e2cf3
♻️ Refactor: `CameraControlBar` 업로드 버튼 영역 높이 수정
chanho0908 fbbecc0
♻️ Refactor: 포토로그 업로드 관련 문자열 리소스 수정
chanho0908 a7f88b4
✨ Feat: `CommonColor`에 `Black` 색상 추가
chanho0908 8b1c408
✨ Feat: 카메라 플래시(Torch) 버튼 UI 구현 및 적용
chanho0908 e4944f6
🔥 Chore: 사용하지 않는 카메라 권한 안내 문자열 제거
chanho0908 1da3655
♻️ Refactor: 토스트 관련 문자열 리소스 네이밍 컨벤션 적용 및 정리
chanho0908 fe31f88
♻️ Refactor: 내 사진이 없을 때 텍스트가 인증샷을 올려보세요!로 변경
chanho0908 b6e49a2
♻️ Refactor: 인증샷 수정 버튼 배경색 변경
chanho0908 9740f5c
♻️ Refactor: 인증샷 업로드 시간 텍스트 색상 변경
chanho0908 4c2fa5c
♻️ Refactor: 인증샷 프레임 radius 수정
chanho0908 c7952fe
♻️ Refactor: 리액션바 shadow 크기 수정
chanho0908 c0273a4
♻️ Refactor: ReactionBar` 아이템 간격 및 패딩 조정
chanho0908 d9699ce
🎨 Style: PhotologDetailPreviewProvider 코드 포맷팅 수정
chanho0908 1f86b09
♻️ Refactor: 댓글 입력 로직 개선 및 공백 검증 강화
chanho0908 5a88f49
✨ Feat: `CommentAnchorFrame` 하단 패딩 커스텀 기능 추가
chanho0908 50f3f6c
♻️ Refactor: 다시찍기 버튼 너비 수정
chanho0908 e7cde7c
♻️ Refactor: `PhotologCardPreview` 접근 제어자 수정
chanho0908 995cf52
♻️ Refactor: `PhotologEditorRoute` 내 `Spacer` 높이를 `101.dp`에서 `103.dp`…
chanho0908 39511e8
♻️ Refactor: 사용하지 않는 `CommentUiModel` 임포트 제거
chanho0908 eb0f518
✨ Refactor: `SwipeableCard` 애니메이션 로직 간소화 및 수치 조정
chanho0908 de3c01f
♻️ Refactor: `dismissDistance`를 화면 너비에 비례하도록 동적 계산 로직 적용
chanho0908 78fc526
✨ Feat: 인증샷 상세 화면 스크롤 적용 및 파일명 변경
chanho0908 082ae06
♻️ Refactor: `CameraPreviewBox` 반응형 레이아웃 대응
chanho0908 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
47 changes: 0 additions & 47 deletions
47
core/design-system/src/main/java/com/twix/designsystem/components/comment/CommentBox.kt
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
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. anchorBottom 기준으로 배치
2. commentTextFieldHeight 만큼 Y축 이동
3. padding 28 만큼 Y축 이동
🚨 문제: 키보드 활성 시 의도하지 않은 여백 발생
키보드가 올라올 때 screenHeight - imeBottom으로 키보드 상단을 계산하면 아래처럼 예상보다 위쪽에 댓글창이 붙는 현상이 발생했음
문제 원인
키보드 높이(imeBottom)에 시스템 네비게이션 바 인셋의 높이가 포함되어 실제 키보드 상단 좌표가 계산보다 더 높게 잡히는 현상이 발생 !!문제 해결
navigationBars인셋을 차감하여 기기별 내비게이션 바 유무와 상관없이 순수 키보드 높이를 구하도록 수정