-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feature/addarticle selectsentence #11
Merged
Merged
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
- 앱 실행마다 가이드 애니메이션이 첫 한번만 보이도록 UserDefault 에 저장
- 선태된 순서를 기반으로 중간에 선택한 사진을 해제할 경우 나머지 사진들이 선택된 순서에 따라 넘버링을 동적으로 업데이트
- SelectedPhotoCell 생성 - 스와이프 페이지 인덱싱 적용 - 네비게이션바에 "다음" 버튼 추가
- AddArticleComponent 에 OrganizingSentenceRepositoryImp 생성 - 잘못된 위치에서 생성된 OrganizingSentenceRepository 코드 제거 - AddArticle 하위 RIB 들이 이를 공유하도록 설정
- UIImageView 에 fetchImage 메서드 completion 파라미터 추가 - UIView 에서 각 코너에 Radius 값을 설정할 수 있는 addRoundedCorner 메서드 추가
- 변수명 소문자 카멜케이스 적용 - 매직리터럴, 매직넘버 enum Metric 으로 분리 - 불필요한 주석 제거, 마크주석 적용 - 개행 컨벤션 적용
- nextButton -> saveButton 으로 네이밍 수정 - 버튼 탭시 데이터 전달
- 텍스트 드래그 시 나타나는 시스템 메뉴 수정 불가 - copy notification을 구독하여 copy 를 탭했을 때 EditSentence 모달을 띄우도록 설정
- EditSentence 모달이 띄워졌을 때 백그라운드 탭 임시 비활성화 - EditSentence 모달 Corner Radius 적용 - attachEditSentence() 중복요청 에러 방지용 조건문 추가
- 마지막 사진에서 문장 추출 후 저장버튼 탭시, 다음 버튼 활성화
Peter1119
approved these changes
Aug 30, 2023
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.
고생하셨습니다~!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
수행사항
@Peter1119