-
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
[TNT-254] QA 반영 #98
Open
FpRaArNkK
wants to merge
32
commits into
develop
Choose a base branch
from
saturday2
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[TNT-254] QA 반영 #98
+721
−227
Conversation
This file contains 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
- 마이페이지 더블 탭시 발생하는 에러 처리 - 에러 이유 : 마이페이지의 알림 설정여부를 비동기로 가져오는 부분이 tca와 충돌 발생(타이밍 안 맞음)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
✨Feat
새로운 기능 구현 (새로운 로직 추가, UI 구현 등)
🛠️Fix
버그 수정 (UI 레이아웃 깨짐, 크래시, 네트워크 오류 등 해결)
♻️Refactor
리팩토링 (코드 구조 변경, 성능 최적화, 가독성 향상 등)
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.
📌 What is the PR?
🪄 Changes
🔥 PR Point
NavStack에 쌓이는 화면 중 특정 화면들, 특정 일부 플로우에서만 제스처가 비활성화되어야하기 때문에,
RefCount 개념을 차용하여 -
현재 표시되고 있는 화면(뒷 화면 + 표시되어지고 있는 화면) 들이 gesture에 대해 비활성화/활성화 인지 체크한 후, onAppear, onDisappear에 상관없이 비활성화를 요구하는 화면 카운트가 기준 아래일 때만 활성화 되도록 진행했습니다.
사용법은 아래와 같습니다.
권한을 받은 경우에만 사진 선택이 가능하며, 권한을 받지 못해 접근이 불가능한 경우 이를 알리는 PhotoLibraryFeature를 작성했습니다.
TCA를 통해 단방향 방식으로 작성하였으며, 각 화면에서 이를 내장하여 Alert 표시, 선택된 아이템 가져오기등 여러 액션이 가능합니다.
사용법은 아래와 같습니다.
1줄, n줄과 상관없이 디자인 시스템에서 요구하는 lineSpacing을 표시하기 위해 제시되는 lineHeight보다 intrinsicSize.height가 작은 경우 해당 텍스트의 frame.height를 lineHeight로 표시하게했습니다. 이를 통해 TButton과 다른 텍스트 관련된 UI QA 내용이 모두 해소되었습니다.
또한 기존 letterSpacing에서 비율로 받은 값을 size에 곱해서 반영하지 않고 있어 해당 부분도 수정했습니다.
🙆🏻 To Reviewers