-
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
Feat [#50] 투명도 퍼센트에 따른 UI 변경 완료 #53
Conversation
|
||
private func setDataBind() { | ||
|
||
} | ||
} |
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.
아래는 코드 패치에 대한 간단한 코드 리뷰입니다. 버그 위험 및 개선 제안 사항을 환영합니다:
transparencyValue
를let
상수로 선언하고 기본값-85
로 설정했습니다.userIntroduction
라벨에label.setTextWithLineHeight
메서드를 사용하여 텍스트를 설정하고, 정렬 옵션.center
를 추가했습니다.emptyTransparencyPercentage
와fullTransparencyPercentage
이미지 뷰의contentMode
를.left
로 변경했습니다.emptyTransparencyPercentage
와fullTransparencyPercentage
이미지 뷰에clipsToBounds
,layer.cornerRadius
,layer.masksToBounds
속성을 추가하여 모서리를 둥글게 처리했습니다.setAddTarget
및setRegisterCell
메서드는 비어 있으므로 삭제할 수 있습니다.setUI
메서드에서backgroundColor
를.donBlack
으로 설정했습니다.transparencyLabel.text
에self.transparencyValue
값을 텍스트로 설정했습니다.
기타 코드 리뷰는 주어진 코드 스니펫에서 파악하기 어렵습니다. 전체 클래스와 관련된 다른 부분도 고려되어야 합니다.
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.
굳!
imageView.contentMode = .left | ||
imageView.clipsToBounds = true | ||
imageView.layer.cornerRadius = 5.adjusted | ||
imageView.layer.masksToBounds = true |
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.
p3
ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ하..ㅜㅜ수고하셨습니다
|
||
let statusBarView = UIView(frame: UIApplication.shared.statusBarFrame) | ||
statusBarView.backgroundColor = UIColor.donBlack // 적절한 색상으로 변경 가능 | ||
view.addSubview(statusBarView) | ||
|
||
let image = ImageLiterals.MyPage.icnMenu | ||
let renderedImage = image.withRenderingMode(.alwaysOriginal) |
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.
이 코드 패치는 MyPageViewController 클래스의 setUI() 메서드에 대한 수정입니다.
개선 제안:
- 뷰의 배경색을 설정하는 줄이 추가되었습니다. donBlack 색상을 사용하고 있는데, 이 색상은 코드에서 어디에 정의되어 있는지 확인해야 합니다.
- 탭 바의 투명성 설정과 네비게이션 바의 속성들이 변경되었습니다. 이러한 변경은 UI 스타일에 따라 다른 의도를 가질 수 있으므로, 주의가 필요합니다. 기존 스타일과 일치하는지 확인해야 합니다.
- 상태 바의 배경색을 설정하는 부분이 추가되었습니다. 마찬가지로 donBlack 색상을 사용하고 있는데, 적절한 색상으로 변경할 수 있습니다.
버그 위험:
장기적인 버그 위험이나 잠재적인 오류는 보이지 않습니다. 그러나 donBlack 색상이 어떻게 정의되어 있는지, 적합한지 확인해야 합니다.
전반적으로 코드 작성자의 의도와 스타일 가이드를 고려해 적절한 수정이 필요할 수 있습니다.
👻 PULL REQUEST
💻 작업한 내용
📸 스크린샷
📟 관련 이슈