-
Notifications
You must be signed in to change notification settings - Fork 2
[FEAT/#148] 스티커 사이즈 조절 기능 추가 #152
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
Conversation
최대 줌 스케일 3->2 배로 변경 이모지 기본 사이즈 64->72로 변경 Co-Authored-By: YeongHoon Song <37678646+0Hooni@users.noreply.github.com>
스티커뷰 서브뷰의 버튼이 프레임에 따라 변경되지 않고 고정사이즈 20으로 변경 Co-Authored-By: YeongHoon Song <37678646+0Hooni@users.noreply.github.com>
Co-Authored-By: YeongHoon Song <37678646+0Hooni@users.noreply.github.com>
Co-Authored-By: seuhong <66902876+hsw1920@users.noreply.github.com>
Co-Authored-By: seuhong <66902876+hsw1920@users.noreply.github.com>
Co-Authored-By: seuhong <66902876+hsw1920@users.noreply.github.com>
Co-Authored-By: seuhong <66902876+hsw1920@users.noreply.github.com>
|
리뷰에 불필요한 diff가 많아서 |
Kiyoung-Kim-57
left a comment
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.
비율 맞춰 사이즈 조절까지!! 너무 좋아요!!👍👍👍
youn9k
left a comment
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.
메소드들이 단일 책임만 하고 있는 것 같아 가독성도 좋고 유지보수하기 편할 것 같습니다!! 고생하셨어요!!
🤔 배경
📃 작업 내역
✅ 리뷰 노트
StickerView.swift
StickerView는 다음과 같은 이벤트를 전달해줍니다CanvasScrollView.swift
StickerView의 이벤트를 전달 받습니다.EditPhotoRoomViewController로 전달해줍니다.EditPhotoRoomViewController.swift
CanvasScrollView로 부터 받은 이벤트를 ViewModel에게 전달해줍니다.로컬 확대/축소 선반영
StickerView.swift
선반영 이후 SoT 적용
StickerView.swift
EditPhotoRoomViewModel.swift
canInteractWithSticker(id: ): 해당 스티커를 소유할 수 있는지 확인해줍니다.mutateSticker(EventHub/Local): 해당 스티커를 이벤트 허브나 로컬에 반영해준다.unlockPreviousSticker(stickerId: ): 이전에 소유하던 스티커의 Lock을 해제해줍니다.lockTappedSticker(id: ): 스티커의 소유권을 나로 변경하고 Lock을 걸어준다.🎨 스크린샷
🚀 테스트 방법