fix: 명함 공유 뷰 수정 (#349) #350

hyun99999 · 2023-01-19T07:54:49Z

🌴 PR 요약

🌱 작업한 브랜치

feature/fix: 명함 공유 뷰 수정 #349
feature/add: Lottie CocoaPods 설치 및 근처 명함 json 파일 추가 #347, feature/add: any,dark/card 색상 추가 #345

Lottie 와 json 파일 및 이미지 에셋을 사용하기 위함. 해당 이슈의 풀리케가 머지된 후에 보시면 좀 더 깔끔하실 거에용

🌱 작업한 내용

명함 공유 뷰 수정.
- 로티 추가.
- UISwitch 에 따라서 UI 변경.
- 바텀시트의 높이가 변경되었습니다.

UIView 를 추가할 때 addSubviews(_:) 메서드를 만들어서 배열로 용이하게 추가하도록 하였습니다.

🚨 참고사항

구현 영상

녹화 영상에서 두번째 시뮬레이터는 활성화된 명함의 경우 명함 공유 바텀시트를 띄우면 활성화가 유지되어야하기 때문에 변수를 수정하여서 적용한 것입니다.
- 활성화 상태에서 로티가 무한반복되도록 요청하여서 반영하였습니당

코드

현재 바텀시트를 등장시킬 때 isActivate 변수값을 false 로 설정하고 있는데 이는 추후에 대응해주면 되겠습니다.
접근 제어자 및 주석과 빈 줄을 수정하였습니다.
UI 를 isActivate 변수에 따라서 처리해야할 때 if 문 대신에 삼항 연산자를 사용하였습니다.
- if 문을 사용하게 되면 else 에서 동일한 components 에 대응해줘야 하기 때문에 줄이 두배 길어지고, 삼항연산자 사용하면 UI components 에 코드가 집중할 수 있다고 생각했습니다.
- if-else 문에서 휴먼에러로 UI components 를 변경하는 코드를 놓치지 않을 수 있습니다.

lazy

lottie 의 경우 활성화되지 않았을 때는 사용되는 것이 불필요하기 때문에 lazy 로 선언해서 lottie image 를 사용하기 전까지 메모리에 올리지 않게 하였습니다.

SwiftLint

SwiftLint 에서 self_in_property_initialization 워닝을 제거하였습니다.
- 아직 선언되지 않은 컴포넌트의 액션 메서드를 적용하기 위해서 addTarget() 에서 self 를 사용하면 이는 예상못할 수 있기 때문에 lazy 하게 만들어야 안전합니다.
출처: https://realm.github.io/SwiftLint/self_in_property_initialization.html

📸 스크린샷

기능	스크린샷
명함 공유 바텀시트

📮 관련 이슈

Resolved: fix: 명함 공유 뷰 수정 #349

…orRelease into feature/#349

- layout 작업 시에 UIView 배열로 뷰에 더하는 함수 생성.

- if 문을 사용하게 되면 줄이 두배 길어지고, 삼항연산자 사용하면 UI components 에 코드가 집중할 수 있다고 생각했습니다. - if-else 문에서 휴먼에러로 변경되는 UI 를 놓치지 않을 수 있습니다.

- 추후에 수정되어야 하기 때문에 주석 설정.

- lazy 로 선언해서 lottie image 를 사용하기 전까지 메모리에 올리지 않게 됨.

- 아직 선언되지 않은 컴포넌트의 액션 메서드를 적용하기 위해서 addTarget() 에서 self 를 사용하면 이는 예상못할 수 있기 때문에 lazy 하게 만들어야 안전합니다.

…ease into feature/#349

dlwns33

수고하셨습니다 선생님! 기기 대응되니까 훨씬 더 보기 좋네요 !

dlwns33 · 2023-01-31T07:50:34Z

...-iOS-forRelease/Sources/ViewControllers/BottomSheet/CardShareBottomSheetViewController.swift

+        cardBackgroundView.translatesAutoresizingMaskIntoConstraints = false
+        NSLayoutConstraint.activate([
+            cardBackgroundView.topAnchor.constraint(equalTo: titleLabel.bottomAnchor, constant: 20.0),
+            cardBackgroundView.leadingAnchor.constraint(equalTo: bottomSheetView.leadingAnchor, constant: 24.0),
+            cardBackgroundView.trailingAnchor.constraint(equalTo: bottomSheetView.trailingAnchor, constant: -24.0),
+            cardBackgroundView.heightAnchor.constraint(equalToConstant: 384.0)
+        ])
+
+        nadaLogoImage.translatesAutoresizingMaskIntoConstraints = false
+        NSLayoutConstraint.activate([
+            nadaLogoImage.topAnchor.constraint(equalTo: cardBackgroundView.topAnchor, constant: 18.0),
+            nadaLogoImage.leadingAnchor.constraint(equalTo: cardBackgroundView.leadingAnchor, constant: 18.0),
+            nadaLogoImage.widthAnchor.constraint(equalToConstant: 84.0),
+            nadaLogoImage.heightAnchor.constraint(equalToConstant: 30.0)
+        ])
+


와우 이거 스냅킷 안쓰시고 하셨군요...

놀랍게도 이 프로젝트는 스냅킷이 인스톨되어있지 않아요 ㅋㅋㅋㅋㅋ 우리가 그때 없이 해보자고 한거 같아여
불편하면 추가해도 좋아요!

…ease into feature/#349 # Conflicts: # NADA-iOS-forRelease.xcodeproj/project.pbxproj

hyun99999 added 22 commits January 18, 2023 21:04

Merge branch 'feature/#347' of https://github.com/TeamNADA/NADA-iOS-F…

2885740

…orRelease into feature/#349

add: 명함 공유 바텀시트를 위한 이미지 에셋 추가 (#349)

3ba5ff1

chore: pod update (#349)

ad541d8

fix: 바텀시트 높이 변경 (#349)

6e103d4

feat: UIView Extension 추가 (#349)

a5beac6

- layout 작업 시에 UIView 배열로 뷰에 더하는 함수 생성.

feat: addSubviews(_:) 함수 적용 (#349)

33a315e

refactor: 접근 제어자 수정 (#349)

cc27170

add: UI Components 추가 (#349)

d3cfebe

delete: FIXME 주석 삭제 (#349)

6ea26dc

fix: 명함 공유 바텀시트의 수정으로 QR Code 의 배경 변경 (#349)

c2c79a8

fix: QR Code 크기 변경 (#349)

d841a23

design: UI layout 수정 (#349)

861ec6d

feat: nearBySwitch 의 액션 메서드 구현 (#349)

e5785b6

refactor: 삼항연산자 사용한 nearBySwitch 액션메서드 리펙토링 (#349)

64c48a4

chore: import Lottie (#349)

0ae2411

feat: 명함 공유 바텀시트가 가지는 변수 설정 (#349)

14e5657

- 추후에 수정되어야 하기 때문에 주석 설정.

add: lottieImage 추가 (#349)

74073da

- lazy 로 선언해서 lottie image 를 사용하기 전까지 메모리에 올리지 않게 됨.

design: lottieImage 레이아웃 설정 (#349)

8c0040c

feat: isActivate 변수에 따른 UI 설정 (#349)

32632a8

feat: nearBySwitch 액션 메서드에 lottieImage 설정 추가 (#349)

857bb48

refactor: SwiftLint (self_in_property_initialization) 워닝 제거 (#349)

e67e4c7

- 아직 선언되지 않은 컴포넌트의 액션 메서드를 적용하기 위해서 addTarget() 에서 self 를 사용하면 이는 예상못할 수 있기 때문에 lazy 하게 만들어야 안전합니다.

hyun99999 added Hyungyu 🐯 현규 교수님 작업 Fix 코드 수정 및 버그, 오류 해결 labels Jan 19, 2023

hyun99999 requested a review from dlwns33 January 19, 2023 07:54

hyun99999 self-assigned this Jan 19, 2023

hyun99999 added 3 commits January 19, 2023 18:56

refactor: 명함의 활성화 유무를 가지고 UI 를 설정해주는 함수로 리펙토링 (#349)

4127d92

Merge branch 'develop' of https://github.com/TeamNADA/NADA-iOS-ForRel…

99c3f33

…ease into feature/#349

chore: Podfile 특정 버전 다운로드하는 옵션 삭제 (#349)

6dafb5f

chore: GoogleService-info.plist 파일이 빠져있어 다시 추가 (#349)

fa04290

dlwns33 approved these changes Jan 31, 2023

View reviewed changes

Merge branch 'develop' of https://github.com/TeamNADA/NADA-iOS-ForRel…

82d5ef4

…ease into feature/#349 # Conflicts: # NADA-iOS-forRelease.xcodeproj/project.pbxproj

hyun99999 merged commit 6e65fb0 into develop Jan 31, 2023

hyun99999 deleted the feature/#349 branch January 31, 2023 09:49

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: 명함 공유 뷰 수정 (#349) #350

fix: 명함 공유 뷰 수정 (#349) #350

hyun99999 commented Jan 19, 2023 •

edited

Loading

dlwns33 left a comment

dlwns33 Jan 31, 2023

hyun99999 Jan 31, 2023

fix: 명함 공유 뷰 수정 (#349) #350

fix: 명함 공유 뷰 수정 (#349) #350

Conversation

hyun99999 commented Jan 19, 2023 • edited Loading

🌴 PR 요약

🚨 참고사항

📸 스크린샷

📮 관련 이슈

dlwns33 left a comment

Choose a reason for hiding this comment

dlwns33 Jan 31, 2023

Choose a reason for hiding this comment

hyun99999 Jan 31, 2023

Choose a reason for hiding this comment

hyun99999 commented Jan 19, 2023 •

edited

Loading