-
Notifications
You must be signed in to change notification settings - Fork 5
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
[Design] #22-리뷰 작성 UI #65
Conversation
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.
캬~~ 쉽지 않은 뷰인데 빠르게 잘 구현해주셨네요...남은 디테일도 화이팅!!
|
||
import BSImagePicker | ||
import SnapKit | ||
import Photos |
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.
p4;
photos는 내장 프레임워크이니 UIKit 위에 써주세용~
func setUI() { | ||
setLineDot(view: addPhotoView, color: UIColor.helfmeTagGray, radius: 8) | ||
} |
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.
잘 한거같은디...왜지?
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.
영린아 멋있다,, 최고
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.
고생하셨습니다 👍🏻
let borderLayer = CAShapeLayer() | ||
borderLayer.strokeColor = color.cgColor | ||
borderLayer.lineDashPattern = [2, 2] | ||
borderLayer.frame = view.bounds | ||
borderLayer.fillColor = nil | ||
borderLayer.path = UIBezierPath(roundedRect: view.bounds, cornerRadius: radius).cgPath |
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.
👍🏻👍🏻 p5;
|
||
private lazy var deleteBtn: UIButton = { | ||
let btn = UIButton() | ||
btn.setImage(ImageLiterals.ReviewWrite.deletePhotoBtn, for: .normal) |
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.
setImage 했을때 이미지 크기는 괜찮았나여?!
setBackgroundImage 했을때랑 비교해서 어떤게 맞는건지 나중에 한번 체크해봐도 괜찮겠네요!! p5;
그 StarSlider 디벨롭에 merge 됐는데 - 한번 Pull 받아서 붙여서 다시 올리면 좋을것같아요!!! |
🔥 Pull requests
⛳️ 작업한 브랜치
👷 작업한 내용
🚨 참고 사항
📸 스크린샷
📟 관련 이슈